D5991: reply to notifications

[Albert Vaca Cintora]

albertvaka accepted this revision.
albertvaka added a comment.
This revision is now accepted and ready to land.


  This is awesome! :D It works great for whatsapp, but it's a pity that doesn't work for Telegram or the default SMS app. Do you know what's the reason?
  
  If we could make this work for SMS, we could remove the code to do that from the telephony plugin and leave just this.
  
  I think we can merge this now, even it doesn't work for every app it's already a great improvement. Do you have commit rights or should I push this on your behalf?

INLINE COMMENTS

> NotificationsPlugin.java:322
> +                    //works for WhatsApp, but not for Telegram
> +                    for(Notification.Action act : statusBarNotification.getNotification().actions) {
> +                        if(act != null && act.getRemoteInputs() != null) {

This contains all the actions from the notification? In a future we could send all the actions with their respective texts, and have everything on the desktop :D

REPOSITORY
  R225 KDE Connect - Android application

BRANCH
  master

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

To: progwolff, #kde_connect, albertvaka
Cc: albertvaka
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20170530/f28e92dc/attachment.html>