From a6884b981d8ebe55eed36decfe87ff16cf0f2720 Mon Sep 17 00:00:00 2001 From: EnixCoda Date: Fri, 14 May 2021 09:42:37 +0800 Subject: [PATCH] fix: skip access token related state if not necessary --- src/components/OAuthWrapper.tsx | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/src/components/OAuthWrapper.tsx b/src/components/OAuthWrapper.tsx index e375e64..e66065c 100644 --- a/src/components/OAuthWrapper.tsx +++ b/src/components/OAuthWrapper.tsx @@ -13,8 +13,15 @@ export function OAuthWrapper({ children }: React.PropsWithChildren<{}>) { const running = useGetAccessToken() const $state = useLoadedContext(SideBarStateContext) + const firstRenderRef = React.useRef(true) React.useEffect(() => { - $state.onChange(running ? 'getting-access-token' : 'after-getting-access-token') + if (running) { + $state.onChange('getting-access-token') + } else if (firstRenderRef.current) { + firstRenderRef.current = false + } else { + $state.onChange('after-getting-access-token') + } }, [running]) // block children rendering on the first render if setting token