From 45f1ca218dc5fd5fed92da6e4aa7a1fb590e5cef Mon Sep 17 00:00:00 2001 From: Kris Urbas <605420+krzysu@users.noreply.github.com> Date: Fri, 4 Aug 2023 16:01:04 +0200 Subject: [PATCH] fix: never throw in useCurrentSession (#466) --- packages/react/src/lifecycle/useCurrentSession.ts | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/packages/react/src/lifecycle/useCurrentSession.ts b/packages/react/src/lifecycle/useCurrentSession.ts index 6deac7b4ff..bbd8751aaa 100644 --- a/packages/react/src/lifecycle/useCurrentSession.ts +++ b/packages/react/src/lifecycle/useCurrentSession.ts @@ -9,7 +9,7 @@ import { useSessionVar, } from '@lens-protocol/api-bindings'; import { WalletData } from '@lens-protocol/domain/use-cases/lifecycle'; -import { invariant, never } from '@lens-protocol/shared-kernel'; +import { invariant } from '@lens-protocol/shared-kernel'; import { useEffect, useRef } from 'react'; import { @@ -149,7 +149,12 @@ export function useCurrentSession(): ReadResult< }; } - never('Previous data must be defined if previous session was WithProfile'); + // shoudn't happen, but just in case, fallback to loading + return { + data: undefined, + error: undefined, + loading: true, + }; } // transitioning from NotAuthenticatedSession to AuthenticatedProfileSession