D9558: Show notifications that are present when pairing

[Matthijs Tijink]

mtijink requested changes to this revision.
mtijink added a comment.
This revision now requires changes to proceed.


  Looks good, generally.

INLINE COMMENTS

> NotificationReceiver.java:83
> +        super.onListenerDisconnected();
> +        for (NotificationListener listener : listeners) {
> +            listener.onListenerConnected(this);

Should be in `onListenerConnected`.

REPOSITORY
  R225 KDE Connect - Android application

REVISION DETAIL
  https://phabricator.kde.org/D9558

To: nicolasfella, #kde_connect, mtijink
Cc: mtijink, #kde_connect, SemperPeritus, ahmedbesbes, daniel.z.tg, jeanv, tfella, aboudhar, seebauer, bugzy, progwolff, MayeulC, menasshock, ach, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20171230/f7e342a2/attachment.html>