D10737: Remove dead code in the ReceiveNotificationsPlugin
Nicolas Fella
noreply at phabricator.kde.org
Thu Feb 22 10:54:59 UTC 2018
nicolasfella created this revision.
nicolasfella added a reviewer: KDE Connect.
nicolasfella added a project: KDE Connect.
nicolasfella requested review of this revision.
REVISION SUMMARY
It doesn't make sense to ask the desktop NotificationsPlugin for it's notifications because they come from the phone anyway. It would make sense to ask the Sendnotifications plugin, but that's not implemented atm.
TEST PLAN
Receiving notifications still works
REPOSITORY
R225 KDE Connect - Android application
BRANCH
sendnotis
REVISION DETAIL
https://phabricator.kde.org/D10737
AFFECTED FILES
src/org/kde/kdeconnect/Plugins/ReceiveNotificationsPlugin/ReceiveNotificationsPlugin.java
To: nicolasfella, #kde_connect
Cc: #kde_connect, adeen-s, SemperPeritus, ahmedbesbes, daniel.z.tg, jeanv, tfella, aboudhar, seebauer, bugzy, MayeulC, menasshock, ach, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20180222/1854ea15/attachment.html>
More information about the KDEConnect
mailing list