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

[David Kahles]

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

(Updated April 6, 2016, 3:53 p.m.)


Status
------

This change has been discarded.


Review request for kdeconnect.


Repository: kdeconnect-kde


Description
-------

Therefore create a component which checks whether the plugin is loaded,
and exposes this information to the DeviceDelegate. The code is mostly
copied from FindMyPhone.qml.


Diffs
-----

  interfaces/notificationsmodel.h 0eb3a4debc1076c7e77b721d00d30a5c72b1ae89 
  plasmoid/package/contents/ui/DeviceDelegate.qml 34dfb1952ed6973c8b3ce5b96fc4799ba37610d7 
  plasmoid/package/contents/ui/NotificationsPluginChecker.qml PRE-CREATION 

Diff: https://git.reviewboard.kde.org/r/127544/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/20160406/3c9ee94a/attachment.html>