[Merge] lp:~phablet-team/history-service/improve_information_events into lp:history-service/staging
Gustavo Pichorim Boiko
gustavo.boiko at canonical.com
Thu Oct 20 07:44:57 UTC 2016
Review: Needs Fixing
Looks good, just has one obsolete comment to remove.
Diff comments:
>
> === modified file 'daemon/historydaemon.h'
> --- daemon/historydaemon.h 2016-10-13 18:21:00 +0000
> +++ daemon/historydaemon.h 2016-10-13 18:21:00 +0000
> @@ -78,7 +78,7 @@
> void updateRoomProperties(const QString &accountId, const QString &threadId, History::EventType type, const QVariantMap &properties, const QStringList &invalidated);
>
> // FIXME: this is a hack. we need proper information event support.
I think this comment is obsolete.
> - void writeInformationEvent(const QVariantMap &thread, const QString &text);
> + void writeInformationEvent(const QVariantMap &thread, History::InformationType type, const QString &subject = QString(), const QString &text = QString());
>
> void writeRoomChangesInformationEvents(const QVariantMap &thread, const QVariantMap &interfaceProperties);
> void writeRolesInformationEvents(const QVariantMap &thread, const Tp::ChannelPtr &channel, const RolesMap &rolesMap);
--
https://code.launchpad.net/~phablet-team/history-service/improve_information_events/+merge/308429
Your team Ubuntu Phablet Team is subscribed to branch lp:~phablet-team/history-service/notify-group-changes-in-conversation.
More information about the Ubuntu-reviews
mailing list