espruino / BangleApps

Bangle.js App Loader (and Apps)
https://banglejs.com/apps
MIT License
478 stars 1.14k forks source link

[banglejs gadgetbridge] message REPLY not handled? #3293

Closed Stuff-etc closed 5 months ago

Stuff-etc commented 5 months ago

Affected hardware version

Bangle 2

Your firmware version

2v21

The bug

Intro

I wanted to add some reply options as mentioned here: https://forum.espruino.com/conversations/373668/ but GB doesn't seem to use them.

How to reproduce:

Installed apps

No response

gfwilliams commented 5 months ago

I think it depends on the app that's being used - not all apps offer reply functionality straight from the notification. I guess WhatsApp might not be one that does?

Stuff-etc commented 5 months ago

I don't think that this is the case here.

I think there is a problem in the BangleJS GB. Do you know if the reply-function was ever successfully tested? If so: can you recommend an app where this should work? Also: Is this the right place for this issue or is there a BJS-GB repo?

gfwilliams commented 5 months ago

Do you know if the reply-function was ever successfully tested?

Yes - I added the code roughly 2 years ago (June 2022) and I'm pretty convinced it worked fine at that point.

However looking back at the code, the implementation is quite involved (see here) so it wouldn't be a total surprise if something got broken.

I've just added a fix that tidies things up and hopefully it'll be merged soon: https://codeberg.org/Freeyourgadget/Gadgetbridge/pulls/3652

But yes, for Gadgetbridge issues it's best to report them in Gadgetbridge itself https://codeberg.org/Freeyourgadget/Gadgetbridge/issues

Stuff-etc commented 5 months ago

thanks, appreciated! Will check it after my vacation!