Fix: TypeError in notification list

src/components/Notifications/NotificationsList.tsx

@@ -261,7 +261,10 @@ export default function NotificationsList({
 
   const handleSubscribeClick = () => {
     const status = isSubscribed;
-    if (status === "NotSubscribed" || status === "SubscribedOnAnotherDevice") {
+    if (!navigator.serviceWorker) {
+      return;
+    }
+    if (["NotSubscribed", "SubscribedOnAnotherDevice"].includes(status)) {
       subscribe();
     } else {
       unsubscribe();
@@ -296,43 +299,44 @@ export default function NotificationsList({
 
   let manageResults: any = null;
 
-  const unsubscribe = () => {
-    navigator.serviceWorker.ready
-      .then(function (reg) {
-        setIsSubscribing(true);
-        reg.pushManager
-          .getSubscription()
-          .then(function (subscription) {
-            subscription
-              ?.unsubscribe()
-              .then(async function (_successful) {
-                const data = {
-                  pf_endpoint: "",
-                  pf_p256dh: "",
-                  pf_auth: "",
-                };
-
-                await request(routes.updateUserPnconfig, {
-                  pathParams: { username: username },
-                  body: data,
-                });
-
-                setIsSubscribed("NotSubscribed");
-                setIsSubscribing(false);
-              })
-              .catch(function (_e) {
-                Error({
-                  msg: t("unsubscribe_failed"),
-                });
-              });
-          })
-          .catch(function (_e) {
-            Error({ msg: t("subscription_error") });
+  const unsubscribe = async () => {
+    try {
+      const reg = await navigator.serviceWorker.ready;
+
+      if (!reg.pushManager) return;
+
+      setIsSubscribing(true);
+
+      const subscription = await reg.pushManager.getSubscription();
+
+      if (subscription) {
+        try {
+          await subscription.unsubscribe();
+
+          const data = {
+            pf_endpoint: "",
+            pf_p256dh: "",
+            pf_auth: "",
+          };
+
+          await request(routes.updateUserPnconfig, {
+            pathParams: { username },
+            body: data,
           });
-      })
-      .catch(function (_e) {
-        Sentry.captureException(_e);
-      });
+
+          setIsSubscribed("NotSubscribed");
+        } catch (e) {
+          Error({ msg: t("unsubscribe_failed") });
+        } finally {
+          setIsSubscribing(false);
+        }
+      } else {
+        setIsSubscribing(false);
+      }
+    } catch (e) {
+      Sentry.captureException(e);
+      Error({ msg: t("subscription_error") });
+    }
   };
 
   async function subscribe() {