Fix logout in deployed environments (#1390) (#1392)

Co-authored-by: Michael Peels <[email protected]>
CDCgov · May 14, 2024 · c704579 · c704579
1 parent 82603c0
commit c704579
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 6 deletions.
diff --git a/apps/modernization-ui/src/logout/logout.module.scss b/apps/modernization-ui/src/logout/logout.module.scss
@@ -29,4 +29,3 @@
  }
  }
 }
-
diff --git a/apps/modernization-ui/src/providers/UserContext.tsx b/apps/modernization-ui/src/providers/UserContext.tsx
@@ -1,6 +1,5 @@
 import React, { ReactNode, useEffect, useReducer } from 'react';
 
-import { Config } from 'config';
 import { User } from 'user';
 
 type InternalState = { status: 'waiting' } | { status: 'ready'; user: User } | { status: 'logout' };
@@ -49,10 +48,7 @@ const UserContextProvider = ({ initial, children }: Props) => {
  // delete cookies
  document.cookie = 'nbs_user=; Max-Age=0; path=/;';
  document.cookie = 'nbs_token=; Max-Age=0; path=/;';
- if (Config.enableLogin) {
- // loading external page will clear state
- window.location.href = '/nbs/logout';
- }
+ window.location.href = '/nbs/logout';
  }
  }, [dispatch, state.status]);
 

diff --git a/apps/modernization-ui/src/shared/header/NavBar.module.scss b/apps/modernization-ui/src/shared/header/NavBar.module.scss
@@ -56,6 +56,7 @@
  font-size: 10pt;
  font-family: Arial, sans-serif;
  padding: 0.15em;
+ cursor: pointer;
  &:focus {
  background-color: $classic-active-navigation-item-background;
  color: $classic-active-navigation-item-color;