Conversation should update when a new message is sent or recieved
Closed, ResolvedPublic

Description

Resolved by D15409 (KDE) and D15360 (Android)

sredman created this task.Aug 30 2018, 4:29 PM
sredman added a comment.EditedAug 30 2018, 11:30 PM

For receiving a message:
My best current thought is to drop support for handling old-style packets in the SMS plugin then make the Android app broadcast both the old and new styles. This supports old desktop apps (by broadcasting the old packet and dropping the new since no desktop plugin advertises support for it) as well as new desktop apps (by broadcasting both the old and new but the desktop ignoring the old)
It would be a good idea to have the desktop SMS plugin not advertise support for the old packet and print a warning in the *telephony* plugin if those packets arrive. If the telephony plugin is updated in the future, the packet could then be dropped entirely

Currernt problem being solved is that a new message does not get processed by the Desktop SMS plugin's "handleBatchMessages" method. This could be hacked and avoided, but why do that when a cleaner way exists?

sredman updated the task description. (Show Details)Sep 10 2018, 3:42 PM
sredman triaged this task as High priority.Sep 16 2018, 7:09 PM
sredman updated the task description. (Show Details)
sredman closed this task as Resolved.Oct 6 2018, 3:48 AM

This feature was added awhile ago in D15409. Whoopsie.