-
Notifications
You must be signed in to change notification settings - Fork 327
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: update react-native-webview [LIVE-12644] #7135
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 5 Ignored Deployments
|
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/react-native-webview@11.26.1 |
bebc796
to
e985f4a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be good to improve the error message, otherwise LGTM
if (supported) { | ||
Linking.openURL(targetUrl); | ||
} else { | ||
console.error(`Don't know how to open URI: ${targetUrl}`); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be a better error message ?
onOpenWindow | URL_UNSUPPORTED : Unable to check if the URL can be opened.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
URL and URI is not the same thing, here we are talking about an URI
I can update this to something like URI not supported: ${targetUrl}
because we are able to check if the URI can be opened but it's just not supported or able to be opened
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
✅ Checklist
npx changeset
was attached.📝 Description
Updates the react-native-webview dependency to the latest version
Keeps all previously patched behaviours:
target="_blank"
links on iOS in the default browserallowInsecureHttps
for recoverKnown bug:
Screen.Recording.2024-06-18.at.17.33.51.mov
Screen.Recording.2024-06-18.at.17.34.56.mov
❓ Context
🧐 Checklist for the PR Reviewers