Signal-android: App does not use android 7.0 system quick reply from the notification drawer

Created on 29 Sep 2016 · 7Comments · Source: signalapp/Signal-Android

I have:

[X] searched open and closed issues for duplicates
[X] read https://github.com/WhisperSystems/Signal-Android/wiki/Submitting-useful-bug-reports

Bug description

Signal app does not utilize new Android system quick reply from notification pull down.

Steps to reproduce

Receive a message
Open notification drawer

Actual result: Quick reply from the notification drawer is not an option.

Expected result: I expect I would be able to reply to messages from the system quick reply like in other messaging applications.

Device info

Device: Nexus 5X
Android version: Android 7.0
Signal version: 3.19.1

feature

Source

walshbm15

👍7

Most helpful comment

Switching to using this API would also have the added benefit of allowing Android Wear users to respond from their watch as it uses the same API.

blunden on 10 Nov 2016

👍5

All 7 comments

Switching to using this API would also have the added benefit of allowing Android Wear users to respond from their watch as it uses the same API.

blunden on 10 Nov 2016

👍5

Can we please move to this on Nougat and above?
Like @blunden mentioned, using the RemoteInput API would also be beneficial to Android Wear users.
It will not affect users on Marshmallow and below anyways if properly implemented and they can continue to enjoy Signal's implementation, which is definitely way better than anything else I've seen until this native method came in.
Also, the new MessagingStyle lets users continue conversations from the notification itself and you can bundle notifications from multiple conversations into one neat summary notification.
It looks like implementing all of this is well documented: Notifications in Android N
Will it be possible to work towards this? This is a beautiful and elegant implementation and with everything ready and waiting to be integrated, it's a shame to miss out on it.

lokesh-krishna on 6 Dec 2016

👍1

Fixed by cce1362.

FeuRenard on 27 Dec 2016

🎉2

@FeuRenard Will this issue be closed once that commit is part of the next update or will it be kept open until the messaging history style is also implemented?
I ask because I wanted to know if a separate issue needs to be created for it or if this can stand in for that as well.

lokesh-krishna on 28 Dec 2016

I would recommend to open a new issue for that, if the mentioned commit doesn't enable your desired feature. Let's wait for the (beta) release.

FeuRenard on 28 Dec 2016

👍1

Does 373a0f952708996c584f92e4e000d270cb22af01 fix this? (It's on the Play Store release 3.26.x.)