-
Notifications
You must be signed in to change notification settings - Fork 106
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
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! 🙏🏻 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6554 +/- ##
==========================================
+ Coverage 49.17% 49.20% +0.03%
==========================================
Files 1554 1554
Lines 32069 32072 +3
Branches 7265 7264 -1
==========================================
+ Hits 15769 15782 +13
+ Misses 16261 16251 -10
Partials 39 39
... and 1 file 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