| nicolasfella |
| KDE Connect |
Update sms app model to use new conversationUpdated signal
Filter incoming messages which belong to a different conversation than the one currently being viewed
See Android-side diff D15360 which adds support for sending live updates when a new message is sent or received
This patch relies on D15360 for Android-side support
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
| smsapp/conversationmodel.cpp | ||
|---|---|---|
| 70 | Please use the new connect syntax. connect(sender, &Sender::signal, receiver, &Receiver::slot) | |
| smsapp/conversationmodel.cpp | ||
|---|---|---|
| 70 | It looks like, because of the way our build system builds the dbus interfaces, the member function definitions are not available to the linker to build the plugin (or, in this case, model). Changing to the new syntax gives lots of undefined reference errors, which is why we rely on runtime connections. Does anyone have a way around this? | |
Implement ContentObserver to listen for incoming/outgoing messages and report those to the remote