Add error comment

Author: MartinBraquet

web/lib/service/web-push.ts

@@ -1,16 +1,16 @@
-import {useEffect} from "react";
-import {urlBase64ToUint8Array} from "common/util/parse";
-import {api} from "web/lib/api";
-import {useUser} from "web/hooks/use-user";
-import {isNativeMobile} from "web/lib/util/webview";
+import {useEffect} from "react"
+import {urlBase64ToUint8Array} from "common/util/parse"
+import {api} from "web/lib/api"
+import {useUser} from "web/hooks/use-user"
+import {isNativeMobile} from "web/lib/util/webview"
 
 const vapidPublicKey = 'BF80q7LrDa4a5ksS2BZrX6PPvL__y0jCNvNqyUzvk8Y4ofTdrS0kRnKfGpClCQAHWmcPHIUmWq8jgQ4ROquSpJQ'
 
 export default function WebPush() {
-  const user = useUser(); // authenticated user
+  const user = useUser() // authenticated user
   const isWeb = typeof window !== 'undefined' && 'serviceWorker' in navigator && !isNativeMobile()
   useEffect(() => {
-    if (!user?.id || !isWeb) return;
+    if (!user?.id || !isWeb) return
     console.log('WebPush', user)
 
     const registerPush = async () => {
@@ -19,14 +19,16 @@ export default function WebPush() {
         .then(async (registration) => {
           console.log('Service worker registered:', registration)
 
-          const permission = await Notification.requestPermission();
+          // May need to ask for permission from a button the user clicks, per this error:
+          // The Notification permission may only be requested from inside a short-running user-generated event handler.
+          const permission = await Notification.requestPermission()
           if (permission !== 'granted') {
             console.log('Notification permission denied')
             return
           }
 
           // Check if already subscribed
-          const existing = await registration.pushManager.getSubscription();
+          const existing = await registration.pushManager.getSubscription()
           if (existing) {
             console.log('Already subscribed:', existing)
             return
@@ -45,7 +47,7 @@ export default function WebPush() {
           console.error('SW registration failed:', err)
           return
         })
-    };
+    }
 
     registerPush()
   }, [user?.id, isWeb])