Fixed: firebase project ID error on login due to missing configs in firebase-messaging-sw

public/firebase-messaging-sw.js

@@ -8,58 +8,69 @@ importScripts('https://www.gstatic.com/firebasejs/8.10.1/firebase-messaging.js')
 const clickActionURL = '/notifications';
 const iconURL = 'img/icons/msapplication-icon-144x144.png';
 
-const firebaseConfig = {
-  apiKey: "",
-  authDomain: "",
-  databaseURL: "",
-  projectId: "",
-  storageBucket: "",
-  messagingSenderId: "",
-  appId: ""
-}
+function handleFirebaseInit() {
+  const firebaseConfig = {
+    apiKey: "",
+    authDomain: "",
+    databaseURL: "",
+    projectId: "",
+    storageBucket: "",
+    messagingSenderId: "",
+    appId: ""
+  }
+
+  // return if configurations are empty
+  if (Object.values(firebaseConfig).some(value => !value)) {
+    return
+  }
 
-// Initialize the Firebase app in the service worker by passing in
-// your app's Firebase config object.
-// https://firebase.google.com/docs/web/setup#config-object
-firebase.initializeApp(firebaseConfig);
+  // Initialize the Firebase app in the service worker by passing in
+  // your app's Firebase config object.
+  // https://firebase.google.com/docs/web/setup#config-object
+  firebase.initializeApp(firebaseConfig);
 
-// Retrieve an instance of Firebase Messaging so that it can handle background
-// messages.
-const messaging = firebase.messaging();
-messaging.onBackgroundMessage(payload => {
-  // Customize notification here
-  const notificationTitle = payload.data.title;
-  const notificationOptions = {
-    body: payload.data.body,
-    icon: iconURL,
-    data: {
-      click_action: clickActionURL
-    }
-  };
-  self.registration.showNotification(notificationTitle, notificationOptions);
+  // Retrieve an instance of Firebase Messaging so that it can handle background
+  // messages.
+  const messaging = firebase.messaging();
+  messaging.onBackgroundMessage(payload => {
+    // Customize notification here
+    const notificationTitle = payload.data.title;
+    const notificationOptions = {
+      body: payload.data.body,
+      icon: iconURL,
+      data: {
+        click_action: clickActionURL
+      }
+    };
+    self.registration.showNotification(notificationTitle, notificationOptions);
 
-  // broadcast background message on FB_BG_MESSAGES so that app can receive that message 
-  const broadcast = new BroadcastChannel('FB_BG_MESSAGES');
-  broadcast.postMessage(payload);
-});
+    // broadcast background message on FB_BG_MESSAGES so that app can receive that message 
+    const broadcast = new BroadcastChannel('FB_BG_MESSAGES');
+    broadcast.postMessage(payload);
+  });
 
-self.addEventListener('notificationclick', event => {
-  event.notification.close();
-  const deepLink = event.notification.data.click_action;
-  event.waitUntil(
-    clients.matchAll({ includeUncontrolled: true, type: 'window' }).then(windowClients => {
-      // Check if the app window is already open
-      for (let client of windowClients) {
-        const clientPath = (new URL(client.url)).pathname;
-        if (clientPath === deepLink && 'focus' in client) {
-          return client.focus();
+  self.addEventListener('notificationclick', event => {
+    event.notification.close();
+    const deepLink = event.notification.data.click_action;
+    event.waitUntil(
+      clients.matchAll({ includeUncontrolled: true, type: 'window' }).then(windowClients => {
+        // Check if the app window is already open
+        for (let client of windowClients) {
+          const clientPath = (new URL(client.url)).pathname;
+          if (clientPath === deepLink && 'focus' in client) {
+            return client.focus();
+          }
         }
-      }
 
-      // If the app window is not open, open a new one
-      if (clients.openWindow) {
-        return clients.openWindow(deepLink);
-      }
-    })
-  );
-});
+        // If the app window is not open, open a new one
+        if (clients.openWindow) {
+          return clients.openWindow(deepLink);
+        }
+      })
+    );
+  });
+}
+
+// wrapping the logic inside function and calling it
+// to provide return statement support
+handleFirebaseInit()