On Android O and above, add link to Android app notification settings in settings menu

RESOLVED FIXED in Firefox 68

Status

()

defect
P3
normal
RESOLVED FIXED
10 months ago
Last month

People

(Reporter: JanH, Assigned: JanH)

Tracking

(Blocks 1 bug)

Trunk
Firefox 68
All
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox63 wontfix, firefox64 wontfix, firefox65 wontfix, firefox68 fixed)

Details

Attachments

(3 attachments)

Notification channels are a new feature on Android O, so to make them more discoverable it might be helpful to put a link to Android's notification settings for our app within the "Notifications" category of our settings.
Assignee: nobody → jh+bugzilla
Jan, are you working on this? It sounds good but not a blocker, so for now I'll mark it as a P3 issue.
Priority: -- → P3
I'd still very much want to do this, but my bug queue is a bit long at the moment - might this be something that Softvision should/could look into?
Flags: needinfo?(sdaswani)
I think P3 is the right priority - unfortunately the SV team has a host of P1s and P2s ahead of this.
Flags: needinfo?(sdaswani)
This would be a nice bit of polish for Fenix. The Chrome notifications settings is much better than what we are doing. Maybe we should inform Bram about this bit of Android UI convention?
Too late to fix in 64. Marking this issue as fix-optional for 65; if you land a patch in nightly and think it's low-risk for beta, please request uplift.

We're not going to change this for now and otherwise Android Studio shows an
error for that file.

Somehow that comment ended up in the wrong file after refactoring.

Starting from Android O, more fine-grained control over which notifications
should be displayed is available through Android's notification channel system.
To aid discoverability, we add a link to the corresponding settings screen from
inside our own settings menu.

Pushed by mozilla@buttercookie.de:
https://hg.mozilla.org/integration/autoland/rev/3e6bf14530c8
Part 0a: Ignore private resource warning. r=VladBaicu
https://hg.mozilla.org/integration/autoland/rev/0cca2f068966
Part 0b: Cleanup javadoc. r=VladBaicu
https://hg.mozilla.org/integration/autoland/rev/a66cab570864
Part 1: Add link to OS app notification settings. r=VladBaicu
You need to log in before you can comment on or make changes to this bug.