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