Review Request 127582: Remove notifications from plasmoid, if the notifications plugin gets disabled

David Kahles david.kahles96 at gmail.com
Thu Apr 7 14:42:44 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127582/
-----------------------------------------------------------

(Updated April 7, 2016, 2:42 p.m.)


Status
------

This change has been marked as submitted.


Review request for kdeconnect.


Changes
-------

Submitted with commit aefa51fa8613cb3add5cbd783849781ac5e50a0e by David Kahles to branch master.


Repository: kdeconnect-kde


Description
-------

Therefore notify the client that we're deleting all notifications.
Otherwise the client wouldn't notice this and the notifications would still
be available in the plasmoid.

This is IHMO a better solution as https://git.reviewboard.kde.org/r/127544/


Diffs
-----

  interfaces/notificationsmodel.h 0eb3a4debc1076c7e77b721d00d30a5c72b1ae89 
  interfaces/notificationsmodel.cpp 0844d03f78f500bfc1399870eb786043c68619d0 
  plugins/notifications/notificationsdbusinterface.h fd36a10885c598ce1611dd4d668c2280b1204c6c 
  plugins/notifications/notificationsdbusinterface.cpp 2e9132b6b98c3466c27222d4f882ed0da2f49465 

Diff: https://git.reviewboard.kde.org/r/127582/diff/


Testing
-------

If i disabled the notification plugin, the notifications stayed in the plasmoid though they were purged in the server, because they were cached.
With this patch, the notifications disappear immediately.


Thanks,

David Kahles

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20160407/b5cc575f/attachment.html>


More information about the KDEConnect mailing list