search

piggz / harbour-amazfish

Amazfit Bip support for SailfishOS
GNU General Public License v3.0
101 stars 51 forks source link

AmazfitNeo notification is using AlertNotificationService #357

Closed jmlich closed 2 months ago

jmlich commented 3 months ago

it seems that max length of message is limited. The specification says 18 characters. The screen shows only "+1" or "+2" anyway.

Code worked for me, but it would be great if @kirbylife can look at it as well.

kirbylife commented 3 months ago

Yes, it seems to work as it would on Android. The only issue I see is that there is no way to enable/disable that feature from Amazfish, I had to go to GadgetBridge to enable notifications and then go back to Amazfish to test. I don't know if I didn't find the way to do it or if it is expected.

jmlich commented 3 months ago

I don't think that there is such configuration in Amazfish. However, it can added.

kirbylife commented 3 months ago

I liked the way you implemented it. Adding the option to enable/disable notifications could go in another PR.