acetheking987/bsky-social-app
1
0
Fork 0
Code Issues Actions Packages Projects Releases 1 Wiki Activity

remove await on dismissBrowser (#9304)

Browse Source
This commit is contained in:
Samuel Newman
2025-10-29 21:23:10 +02:00
committed by GitHub
parent 2d626f7bbf
commit cfa2534ccb
1 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/lib/hooks/useIntentHandler.ts
+7 -4
View File
@@ -5,7 +5,7 @@ import * as WebBrowser from 'expo-web-browser'
import {useOpenComposer} from '#/lib/hooks/useOpenComposer' import {useOpenComposer} from '#/lib/hooks/useOpenComposer'
import {logger} from '#/logger' import {logger} from '#/logger'
import {isNative} from '#/platform/detection' import {isIOS, isNative} from '#/platform/detection'
import {useSession} from '#/state/session' import {useSession} from '#/state/session'
import {useCloseAllActiveElements} from '#/state/util' import {useCloseAllActiveElements} from '#/state/util'
import { import {
@@ -34,9 +34,12 @@ export function useIntentHandler() {
React.useEffect(() => { React.useEffect(() => {
const handleIncomingURL = async (url: string) => { const handleIncomingURL = async (url: string) => {
if (isNative) { if (isIOS) {
// Close in-app browser if it's open // Close in-app browser if it's open (iOS only)
await WebBrowser.dismissBrowser().catch(() => {}) // TEMP: promise never resolves if the browser is not open, so don't await
// https://github.com/expo/expo/issues/40710
// add the await back when possible since it's needed to fix the IAB share bug -sfn
/* await */ WebBrowser.dismissBrowser().catch(() => {})
} }
const referrerInfo = Referrer.getReferrerInfo() const referrerInfo = Referrer.getReferrerInfo()