#4140: Changed the approach to reload notifications if auth token is changed.

Author: luizrrodrigues

src/shared/components/Header/index.jsx

@@ -2,8 +2,6 @@ import _ from 'lodash';
 import React, { useState, useEffect } from 'react';
 import PT from 'prop-types';
 import { config } from 'topcoder-react-utils';
-import { isTokenExpired } from 'tc-accounts';
-import { goToLogin } from 'utils/tc';
 import Logo from 'assets/images/tc-logo.svg';
 
 let TopNavRef;
@@ -52,17 +50,18 @@ const Header = ({
 
   useEffect(() => {
     setPath(window.location.pathname);
+  }, []);
 
-    if (auth.authenticating) return;
-
-    // Check auth token, go to login page if expired
-    if (auth.tokenV3 && isTokenExpired(auth.tokenV3)) {
-      goToLogin('community-app-main');
-      return;
-    }
+  /*
+   * Reload notificaitons if token was changed
+   * This prevent to use expired token in API call
+   */
+  if (auth) {
+    useEffect(() => {
+      loadNotifications(auth.tokenV3);
+    }, [auth.tokenV3]);
+  }
 
-    loadNotifications(auth.tokenV3);
-  }, []);
   if (TopNavRef) {
     return (
       <div>

src/shared/containers/TopcoderHeader.js

@@ -48,7 +48,9 @@ function mapStateToProps(state) {
     notifications: (state.notifications
       && state.notifications.items
       && [...state.notifications.items]) || [],
-    auth: state.auth,
+    auth: {
+      ...state.auth,
+    },
   };
 }