Merge pull request #174 from k2maan/fixFirebaseConfigError

Fixed: firebase error on login if app is not configured for notifications support
hotwax · Sep 22, 2023 · 2712314 · 2712314
2 parents 48ad689 + 60cc38c
commit 2712314
Showing 1 changed file with 10 additions and 7 deletions.
diff --git a/src/components/Login.ts b/src/components/Login.ts
@@ -63,13 +63,16 @@ export default defineComponent({
       try {
         await context.login({ token, oms })
 
-        // initialising and connecting firebase app for notification support
-        await initialiseFirebaseApp(
-          noitificationContext.appFirebaseConfig,
-          noitificationContext.appFirebaseVapidKey,
-          noitificationContext.storeClientRegistrationToken,
-          noitificationContext.addNotification,
-        )
+        // check if firebase configurations are there
+        if (noitificationContext.appFirebaseConfig) {
+          // initialising and connecting firebase app for notification support
+          await initialiseFirebaseApp(
+            noitificationContext.appFirebaseConfig,
+            noitificationContext.appFirebaseVapidKey,
+            noitificationContext.storeClientRegistrationToken,
+            noitificationContext.addNotification,
+          )
+        }
 
         this.router.push('/')
       } catch (error) {