polito / students-app

Politecnico di Torino's official mobile application for students
European Union Public License 1.2
156 stars 17 forks source link

Notifications do not work on Android 13/14 #336

Closed MarcAnt01 closed 11 months ago

MarcAnt01 commented 1 year ago

Describe the bug Notifications by default are not working on Android, this bug is also present in the old app.

To Reproduce Do a fresh install of the app, wait for a notification I guess😂

Expected behavior Get notifications

Screenshots Not needed

Smartphone (please complete the following information):

Additional context A workaround for this is to go in system settings and enable notifications, at this point notifications work. Maybe the app has to request to use push notifications (?)

Are you willing to contribute to this issue? [Yes/No] No

lpezzolla commented 1 year ago

Thank you for getting in touch. We are investigating this, it is not a general problem of Android users, it may be related to your specific device or the fact you had a previous version of polito students w/o notification already installed

MarcAnt01 commented 1 year ago

@lpezzolla I doubt that it's a device specific problem, since Google Pixel devices are basically running pure Android and do not have stuff that breaks notifications like chinese Android Roms like oxygen OS, MIUI, EMUI etc. Furthermore in the public group infopolito, there have been several reports about the misfunction of notifications. What I think is that the app doesn't show the banner to allow notifications to be shown, I haven't been using Android for a long time (since I used iPhones for years), but I have noticed that some apps do show a banner basically asking to show notifications that enables that setting I'm editing manually (maybe it's something specific to latest Android versions?), it could be possible that polito app doesn't do this, because I haven't seen it after fresh install

lpezzolla commented 1 year ago

This seems to be related to the newly introduced permission management for notifications in API 33, only affecting Android 13/14 users.

https://developer.android.com/develop/ui/views/notifications/notification-permission

We will fix this in the upcoming weeks