Closed Bug 1278851 Opened 5 years ago Closed 5 years ago
Enable service worker notifications everywhere except for non-release B2G
Already r+'ed by baku in bug 1278435, comment 10. Moving the patch out into a separate bug because this one needs to go through the uplift process and land everywhere. Requesting tracking for 48 and 49 because we want to ship push in Android 48. Without this, a service worker could receive pushes, but not show any notifications.
Attachment #8761213 - Flags: review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/5a5df0f767ed Enable service worker notifications everywhere except for non-release B2G. r=baku
Comment on attachment 8761213 [details] [diff] [review] notifsEverywhere.patch Approval Request Comment [Feature/regressing bug #]: Android push notifications. [User impact if declined]: Service workers won't be able to show notifications to the user on Android release builds. [Describe test coverage new/current, TreeHerder]: Covered by existing tests. [Risks and why]: Low risk, since service worker notifications were already available in non-release builds. [String/UUID change made/needed]: None.
Comment on attachment 8761213 [details] [diff] [review] notifsEverywhere.patch OK, let's ship that! Thanks Should be in 48 beta 2
You need to log in before you can comment on or make changes to this bug.