Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notifications setting cannot be changed #544

Open
mvl22 opened this issue Jan 21, 2024 · 5 comments
Open

Notifications setting cannot be changed #544

mvl22 opened this issue Jan 21, 2024 · 5 comments

Comments

@mvl22
Copy link
Member

mvl22 commented Jan 21, 2024

We've had a report as follows; two screenshots below. Can we try to get an exact description of the problem.

'Permission needs granting in device settings' I wasn't getting this error message before. I had to uninstall the app and reinstall it.

The other photo is the app setting on my phone. It doesn't let you change the setting, as I think it's been installed as a system app on my phone, even though I downloaded from the Play Store as normal and I don't know why.

attachment-0002

attachment-0003

(NB I've blanked out the location that was shown as that's not relevant.)

@HilaryN
Copy link
Contributor

HilaryN commented Jan 28, 2024

The "Permission needs granting in device settings" message is due to WRITE_EXTERNAL_STORAGE permission being deprecated in Android 13. It means map tiles can't be cached and should be fixed in the PR I have submitted. (#543 ).
I'm not sure about the notifications issue.

@HilaryN
Copy link
Contributor

HilaryN commented Mar 18, 2024

This is what shows on my phone for the app Notifications setting. The app doesn't send notifications so it shouldn't be an issue.

Screenshot_20240318-193323

@HilaryN HilaryN closed this as completed Mar 18, 2024
@mvl22
Copy link
Member Author

mvl22 commented Mar 18, 2024

Doesn’t the blog send a notification?

If not, if there are no notifications, why does this setting exist?

@oliverlockwood
Copy link
Member

The blog does not send a notification.

During LiveRide, notifications are used to show the most recent instruction text. It is my understanding that it is mandatory in Android to have notifications to allow the background service to keep running eg if you have your phone screen switched off while riding.

So I am not convinced by the closing of this issue in this way.

@mvl22 mvl22 reopened this Mar 19, 2024
@HilaryN
Copy link
Contributor

HilaryN commented Mar 20, 2024

Ah, I hadn't realised that notification could be switched off, apologies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants