Update to API 33 - Githubissues

BasicAirData / GPSLogger

A GPS logger for Android mobile devices

http://www.basicairdata.eu/projects/android/android-gps-logger/

GNU General Public License v3.0

405 stars 126 forks source link

Update to API 33 #212

Closed GrazianoCapelli closed 10 months ago

GrazianoCapelli commented 1 year ago

We must update the app to API 33. It is mandatory by Google, we cannot publish app updates without switch to the new API.

To provide users with a safe and secure experience, Google Play requires all apps to meet target API level requirements before Aug 31, 2023.

[x] Update of the dependencies
[x] Manage the new Notification Runtime Permission for Android 13+, that is necessary to display/update the notification
[x] ~~Add a warning in case the notification permission is denied (new string)~~

GrazianoCapelli commented 11 months ago

When the user denies the Notification permission in Android 13+, the app keeps working well because the foreground service works as usual keeping up the background recording. The only difference is that the user cannot see the notification on the upper notification bar. In this case the user can see that the app is working in background only by looking into the Task Manager:

I'm thinking that maybe we could avoid to add to the GPS-FIX tab a warning that suggests the users to allow the Notifications Permission...

GrazianoCapelli commented 10 months ago

I think we could ask the permission (as implemented in commit d6fc180) without show any warning in case the user denies the permission. The foreground service keeps working in both cases. Other apps implemented this behaviour; we can collect feedback and in case we could decide to add the warning in a future update.