FIX: Don't support push notification if web notification is not enabled.

* In Firefox, web notifications can be disabled by setting `dom.webnotifications.enabled` to `false`.
discourse · Mar 21, 2018 · 8e217c4 · 8e217c4
1 parent aac3e3d
commit 8e217c4
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/assets/javascripts/discourse/lib/push-notifications.js.es6 b/assets/javascripts/discourse/lib/push-notifications.js.es6
@@ -24,6 +24,7 @@ function userAgentVersionChecker(agent, version, mobileView) {
 export function isPushNotificationsSupported(mobileView) {
   if (!(('serviceWorker' in navigator) &&
      (ServiceWorkerRegistration &&
+     (typeof(Notification) !== "undefined") &&
      ('showNotification' in ServiceWorkerRegistration.prototype) &&
      ('PushManager' in window)))) {