From 52dc5e41a520cc02f61894991c5c6f8ea1f3695e Mon Sep 17 00:00:00 2001 From: Max Leiter Date: Sat, 2 Apr 2022 00:46:39 -0700 Subject: [PATCH] client: remove signout() function from useSignedIn We now use the middleware-implemented /signout route --- client/components/header/index.tsx | 3 +-- client/lib/hooks/use-signed-in.ts | 10 ++-------- 2 files changed, 3 insertions(+), 10 deletions(-) diff --git a/client/components/header/index.tsx b/client/components/header/index.tsx index dfb99714..3990726e 100644 --- a/client/components/header/index.tsx +++ b/client/components/header/index.tsx @@ -35,7 +35,7 @@ const Header = () => { const [expanded, setExpanded] = useState(false) const [, setBodyHidden] = useBodyScroll(null, { scrollLayer: true }) const isMobile = useMediaQuery('xs', { match: 'down' }) - const { signedIn: isSignedIn, signout } = useSignedIn() + const { signedIn: isSignedIn } = useSignedIn() const userData = useUserData(); const [pages, setPages] = useState([]) const { setTheme, resolvedTheme } = useTheme() @@ -93,7 +93,6 @@ const Header = () => { name: 'sign out', icon: , value: 'signout', - // onClick: signout, href: '/signout' }, ...defaultPages diff --git a/client/lib/hooks/use-signed-in.ts b/client/lib/hooks/use-signed-in.ts index 75129d12..f55ffab4 100644 --- a/client/lib/hooks/use-signed-in.ts +++ b/client/lib/hooks/use-signed-in.ts @@ -1,6 +1,6 @@ import Cookies from "js-cookie" import { useRouter } from "next/router" -import { useEffect, useState } from "react" +import { useEffect } from "react" import useSharedState from "./use-shared-state" const useSignedIn = () => { @@ -16,12 +16,6 @@ const useSignedIn = () => { Cookies.set("drift-token", token) } - const signout = () => { - setSignedIn(false) - Cookies.remove("drift-token") - router.push("/") - } - useEffect(() => { if (token) { setSignedIn(true) @@ -30,7 +24,7 @@ const useSignedIn = () => { } }, [setSignedIn, token]) - return { signedIn, signin, token, signout } + return { signedIn, signin, token } } export default useSignedIn