diff --git a/scm-ui/ui-api/src/login.ts b/scm-ui/ui-api/src/login.ts index b4daab0eb4..3ef4425e59 100644 --- a/scm-ui/ui-api/src/login.ts +++ b/scm-ui/ui-api/src/login.ts @@ -104,23 +104,22 @@ export const useLogout = () => { const link = useIndexLink("logout"); const reset = useReset(); - const { mutate, isLoading, error, data } = useMutation(() => - apiClient.delete(link!).then(r => (r.status === 200 ? r.json() : {})) + const { mutate, isLoading, error, data } = useMutation( + () => apiClient.delete(link!).then(r => (r.status === 200 ? r.json() : {})), + { + onSuccess: response => { + if (response?.logoutRedirect) { + window.location.assign(response.logoutRedirect); + } + reset(); + } + } ); const logout = () => { mutate({}); }; - useEffect(() => { - if (data?.logoutRedirect) { - window.location.assign(data.logoutRedirect); - } - if (data) { - reset(); - } - }, [data, reset]); - return { logout: link && !data ? logout : undefined, isLoading, diff --git a/scm-ui/ui-components/src/__snapshots__/storyshots.test.ts.snap b/scm-ui/ui-components/src/__snapshots__/storyshots.test.ts.snap index 3decdfae60..b89821b4cd 100644 --- a/scm-ui/ui-components/src/__snapshots__/storyshots.test.ts.snap +++ b/scm-ui/ui-components/src/__snapshots__/storyshots.test.ts.snap @@ -3734,24 +3734,11 @@ Array [ className="Breadcrumbstories__Wrapper-sc-1eq8sgz-0 hTbphm" >
-
- - - -

-
- - - -