[Merge] lp:~phablet-team/history-service/include-sender-in-info-events into lp:history-service
Tiago Salem Herrmann
tiago.herrmann at canonical.com
Fri Sep 16 12:24:50 UTC 2016
Review: Needs Fixing
Diff comments:
>
> === modified file 'Ubuntu/History/historyeventmodel.h'
> --- Ubuntu/History/historyeventmodel.h 2016-09-16 12:04:11 +0000
> +++ Ubuntu/History/historyeventmodel.h 2016-09-16 12:04:12 +0000
> @@ -49,7 +49,8 @@
> CallMissedRole,
> CallDurationRole,
> RemoteParticipantRole,
> - LastEventRole
> + LastEventRole,
can you move LastEventRole to be the last item in the enum.
That's used by inherited models to avoid overwriting roles from base classes.
> + SubjectAsAliasRole
> };
>
> explicit HistoryEventModel(QObject *parent = 0);
--
https://code.launchpad.net/~phablet-team/history-service/include-sender-in-info-events/+merge/305941
Your team Ubuntu Phablet Team is subscribed to branch lp:~phablet-team/history-service/group-invitation-info-event.
More information about the Ubuntu-reviews
mailing list