D16599: [Desktop] Make message syncronization interface capable of handling future changes
Simon Redman
noreply at phabricator.kde.org
Thu Nov 1 20:31:13 GMT 2018
sredman created this revision.
sredman added a reviewer: KDE Connect.
Herald added a project: KDE Connect.
Herald added a subscriber: kdeconnect.
sredman requested review of this revision.
REVISION SUMMARY
Add "event" field to ConversationMessage
Update packet type with proper field names and commenting
Future "proof" SmsPlugin against new event types
TEST PLAN
- Install the corresponding Android-side patch
- Verify that it is possible to synchronize messages the same as it was before
REPOSITORY
R224 KDE Connect
BRANCH
messaging-future-view
REVISION DETAIL
https://phabricator.kde.org/D16599
AFFECTED FILES
interfaces/conversationmessage.cpp
interfaces/conversationmessage.h
plugins/sms/smsplugin.cpp
plugins/sms/smsplugin.h
To: sredman, #kde_connect
Cc: kdeconnect, skymoore, wistak, dvalencia, rmenezes, julioc, Leptopoda, timothyc, jdvr, yannux, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, tctara, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20181101/d9532900/attachment.html>
More information about the KDEConnect
mailing list