fix(messaging): check if notifications are supported before checking …

…or requesting permission (#701)
capawesome-team · Dec 16, 2024 · cabf5ac · cabf5ac
1 parent 6bb49b5
commit cabf5ac
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/.changeset/cuddly-bears-return.md b/.changeset/cuddly-bears-return.md
@@ -0,0 +1,5 @@
+---
+'@capacitor-firebase/messaging': patch
+---
+
+fix(web): check if notifications are supported before checking or requesting permission
diff --git a/packages/messaging/src/web.ts b/packages/messaging/src/web.ts
@@ -44,6 +44,12 @@ export class FirebaseMessagingWeb
   }
 
   public async checkPermissions(): Promise<PermissionStatus> {
+    const isSupported = await isSupportedInWeb();
+    if (!isSupported) {
+      return {
+        receive: 'denied',
+      };
+    }
     const receive = this.convertNotificationPermissionToPermissionState(
       Notification.permission,
     );
@@ -53,6 +59,12 @@ export class FirebaseMessagingWeb
   }
 
   public async requestPermissions(): Promise<PermissionStatus> {
+    const isSupported = await isSupportedInWeb();
+    if (!isSupported) {
+      return {
+        receive: 'denied',
+      };
+    }
     const notificationPermission = await Notification.requestPermission();
     const receive = this.convertNotificationPermissionToPermissionState(
       notificationPermission,