-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Test handle universal link #8793
Conversation
#no-changelog |
Coverage ReportAffected SDKsNo changes between base commit (6a12566) and head commit (e2f8ba3). Test Logs
|
Binary Size ReportAffected SDKsNo changes between base commit (6a12566) and head commit (e2f8ba3). Test Logs
|
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.
Thanks for investigating!
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.
LGTM with Changelog updates and green CI
Add unit test to check DynamicLinks.dynamicLinks().handleUniversalLink(url) to return YES for Links of the format https://*.page.link/test
Related Issue : #8786