-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
Add Linking examples in rn-tester #30547
Add Linking examples in rn-tester #30547
Conversation
8b92f55
to
ba47395
Compare
Can you add a screenshot? |
Looks like the JSC test is failing, going to re-run the test and see if it's an infra error |
Looks like the failing iOS test is due to You should be able to run the tests locally using:
|
@lunaleaps has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
@utkarsh-dixit Is this only for iOS? |
@@ -2,6 +2,14 @@ | |||
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd"> | |||
<plist version="1.0"> | |||
<dict> | |||
<key>LSApplicationQueriesSchemes</key> |
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.
Do we need to do something equivalent for Android
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.
Ah I see, so this example was already working for Android but now is adding support for iOS? Should we rename the example component?
@lunaleaps merged this pull request in b26f277. |
Summary
Added examples to RNTester for the Linking API.
Changelog
[General] [Added] - Added example for openSettings() for Linking API.
[General] [Added] - Added LSApplicationQueriesSchemes in info.plist with entries tel, telprompt, http, fb, geo
Test Plan