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