Merge pull request #645 from rtucker-mozilla/master

Looking to reduce calls to the api for unauthenticated users

Author: rtucker-mozilla

src/components/access_group/AccessGroupInvitationNotification.vue

@@ -81,7 +81,12 @@ export default {
         path: 'selfInvitations/get',
       });
 
-      return data.map((invite) => new GroupInvitationViewModel(invite));
+      if(data.map){
+        return data.map((invite) => new GroupInvitationViewModel(invite));
+      } else {
+        return {}
+      }
+
     },
     async acceptInvitation(currentInvitation) {
       return await this.accessGroupApi.execute({

src/store/index.js

@@ -8,7 +8,12 @@ import user from './user.store';
 import scope from './scope.store';
 import features from './features.store';
 
+let shouldFetch = true;
+
 async function fetchUser(commit) {
+  if (shouldFetch === false){
+    return
+  }
   try {
     const fetchPolicy = 'network-only';
     const { data } = await client.query({
@@ -19,14 +24,21 @@ async function fetchUser(commit) {
     commit('setUser', data.profile);
     return true;
   } catch (e) {
-    const {
-      graphQLErrors: [{ message }],
-    } = e;
-    if (message === 'wait_for_profile') {
-      console.log('creating');
-    }
-    commit('setUser', { loggedIn: true });
-    return false;
+    try {
+      const {
+        graphQLErrors: [{ message }],
+      } = e;
+      if (message === 'wait_for_profile') {
+        console.log('creating');
+      }
+      commit('setUser', { loggedIn: true });
+      return false;
+      }
+      catch(e) {
+        // Probably getting a 403 for authentication
+        shouldFetch = false
+        return false
+      }
   }
 }
 
@@ -101,4 +113,4 @@ export async function resolvePromisesSerially(promises, resolvers) {
     console.error(e.message);
     throw new Error(e.message);
   }
-}
+}