Improved: used replace method in place of href for logout cases as well

hotwax · Oct 29, 2024 · 43ff78a · 43ff78a
1 parent 24ed420
commit 43ff78a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/components/DxpLogin.vue b/src/components/DxpLogin.vue
@@ -50,7 +50,7 @@ const error = ref({
 
 onMounted(async () => {
   if (!Object.keys(route.query).length) {
-    window.location.href = context.appLoginUrl
+    window.location.replace(context.appLoginUrl)
     return
   }
 
@@ -73,7 +73,7 @@ async function handleUserFlow(token: string, oms: string, expirationTime: string
     console.error('User token has expired, redirecting to launchpad.')
     error.value.message = 'User token has expired, redirecting to launchpad.'
     const redirectUrl = window.location.origin + '/login' // current app URL
-    window.location.href = `${context.appLoginUrl}?isLoggedOut=true&redirectUrl=${redirectUrl}`
+    window.location.replace(`${context.appLoginUrl}?isLoggedOut=true&redirectUrl=${redirectUrl}`)
     return
   }
 
@@ -116,7 +116,7 @@ async function handleUserFlow(token: string, oms: string, expirationTime: string
 }
 
 function goToLaunchpad() {
-  window.location.href = `${process.env.VUE_APP_LOGIN_URL}`
+  window.location.replace(process.env.VUE_APP_LOGIN_URL)
 }
 </script>
 

diff --git a/src/components/DxpUserProfile.vue b/src/components/DxpUserProfile.vue
@@ -65,7 +65,7 @@ const logout = () => {
   
   appState.dispatch('user/logout').then(() => {
     const redirectUrl = window.location.origin + '/login'
-    window.location.href = `${appLoginUrl}?isLoggedOut=true&redirectUrl=${redirectUrl}`
+    window.location.replace(`${appLoginUrl}?isLoggedOut=true&redirectUrl=${redirectUrl}`)
   })
 }