-
Notifications
You must be signed in to change notification settings - Fork 105
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
fix: [IOPID-2547] Fix unhandled error on Linking.openUrl
#6554
base: master
Are you sure you want to change the base?
Conversation
Linking.openUrl
Linking.openUrl
Affected stories
|
@@ -36,7 +38,9 @@ const CieIdNotInstalled = ({ isUat }: CieIdNotInstalledProps) => { | |||
android: isUat ? CIE_ID_ANDROID_COLL_LINK : CIE_ID_ANDROID_LINK, | |||
default: "" | |||
}) | |||
); | |||
).catch(() => { |
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.
Instead of defining a catch
here why not using openWebUrl
utility function? (here the code)
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.
Thank you for this suggestion! Addressed here
@mariateresaventura copy review needed! 🙏🏻 |
Done! |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6554 +/- ##
==========================================
+ Coverage 48.42% 49.28% +0.86%
==========================================
Files 1488 1553 +65
Lines 31617 31986 +369
Branches 7669 7288 -381
==========================================
+ Hits 15311 15765 +454
+ Misses 16238 16182 -56
+ Partials 68 39 -29
... and 1737 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Jira Pull Request LinkThis Pull Request refers to the following Jira issue IOPID-2547 |
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
Short description
This PR handles any potential errors when attempting to open the store using
Linking.openUrl
.List of changes proposed in this pull request
toast message
to notify the user of the error in case the store cannot be opened.Screen