Skip to content

Commit e5620dd

Browse files
dBianchiidbianchiijuliusmarminge
authored
Expo > Don't router.replace() on dismiss web view (#1079)
Co-authored-by: dbianchii <[email protected]> Co-authored-by: juliusmarminge <[email protected]>
1 parent 51b0657 commit e5620dd

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

apps/expo/src/utils/auth.tsx

+7-3
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,14 @@ export const signIn = async () => {
1414
redirectTo,
1515
);
1616

17-
if (result.type !== "success") return;
17+
if (result.type !== "success") return false;
1818
const url = Linking.parse(result.url);
1919
const sessionToken = String(url.queryParams?.session_token);
20-
if (!sessionToken) return;
20+
if (!sessionToken) throw new Error("No session token found");
2121

2222
setToken(sessionToken);
23+
24+
return true;
2325
};
2426

2527
export const useUser = () => {
@@ -32,7 +34,9 @@ export const useSignIn = () => {
3234
const router = useRouter();
3335

3436
return async () => {
35-
await signIn();
37+
const success = await signIn();
38+
if (!success) return;
39+
3640
await utils.invalidate();
3741
router.replace("/");
3842
};

0 commit comments

Comments
 (0)