fix: PushManager does not exist on older devices (#744)

Author: Jonasdoubleyou

src/lib/WebPush.ts

@@ -68,7 +68,7 @@ export async function subscribeUserToPush(serverKey: string) {
     log('WebPush', 'Registering service worker');
 
     const sw = await getServiceWorker();
-    if (!sw) {
+    if (!sw || !sw.pushManager) {
         log('WebPush', 'No service worker present');
         return;
     }
@@ -153,12 +153,6 @@ export function useWebPush() {
             return;
         }
 
-        // Do we have a service worker running?
-        if (!getServiceWorker()) {
-            setStatus('not-supported');
-            return;
-        }
-
         // Did the user deny us permissions? If yes,
         // we cannot ask again :/
         if (Notification.permission === 'denied') {
@@ -180,6 +174,12 @@ export function useWebPush() {
                 return;
             }
 
+            // Older Safari is not spec compliant and does not have .pushManager:
+            if (!sw.pushManager) {
+                setStatus('not-supported');
+                return;
+            }
+
             const subscription = await sw.pushManager.getSubscription();
             if (!subscription) {
                 setStatus('not-subscribed');
@@ -258,7 +258,7 @@ export function useWebPush() {
         try {
             const sw = await getServiceWorker();
             if (sw) {
-                const subscription = await sw.pushManager.getSubscription();
+                const subscription = await sw.pushManager?.getSubscription();
                 await subscription?.unsubscribe();
             }