Merge pull request #236 from AAMasters/develop

BUG FIXED: Master App was deleting ALL localstorage when logging out …

client/src/auth/index.js

@@ -213,7 +213,9 @@ class Auth {
 
   logout () {
     // Clear access token and ID token from local storage
-    window.localStorage.clear()
+    window.localStorage.removeItem('access_token')
+    window.localStorage.removeItem('user')
+    window.localStorage.removeItem('name')
     deleteCookie('ajs_anonymous_id')
     deleteCookie('ajs_user_id')
     deleteCookie('current_tenant')

client/src/graphql/apollo.js

@@ -15,7 +15,9 @@ const authRetryLink = onError(
       // User access token has expired
       // console.log('authLink: ', graphQLErrors) // check for error message to intercept and resend with Auth0 access token
       if (graphQLErrors[0].message === 'Context creation failed: jwt expired') {
-        window.localStorage.clear()
+        window.localStorage.removeItem('access_token')
+        window.localStorage.removeItem('user')
+        window.localStorage.removeItem('name')
         location.reload()
       } else if (graphQLErrors[0].message === 'Not logged in') {
         // We assume we have auth0 access token needed to run the async request

client/src/views/layout/header/index.js

@@ -76,7 +76,9 @@ class Header extends Component {
     let { onTop, mobileOpen, openedMenu } = this.state
     if (window.localStorage.getItem('access_token')) {
       if (jwtDecode(window.localStorage.getItem('access_token')).exp < new Date().getTime() / 1000) {
-        window.localStorage.clear()
+        window.localStorage.removeItem('access_token')
+        window.localStorage.removeItem('user')
+        window.localStorage.removeItem('name')
         window.location.reload()
       }
     }
@@ -86,7 +88,7 @@ class Header extends Component {
       if (authenticated && !(this.state.user !== undefined && this.state.user !== null)) {
         return
       }
-      let cssChildren = submenus.length === 0 ? null : 'hasChildren';
+      let cssChildren = submenus.length === 0 ? null : 'hasChildren'
       return (
         <li
           onMouseEnter={() => this.toggleMenuOpen(index, bigScreen)}