-
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
Linking module fails when user has uninstalled Mail.app #13114
Comments
We're cutting down on the number of outstanding issues, in order to allow us to focus. I'm closing this issue because it has been open for over 60 days with no activity. If you think it should still be opened let us know why. |
This is still an issue - please reopen |
@lprhodes do you happen to have more information about the cause of the crash? Any logs or specific error message? This sounds like something that can be easily fixed within Linking by checking if the system supports opening the URL. |
I'll follow up with the crash report later but basically messages:// is returning that it's supported so you then go to open it but it crashes because it's been deleted. It wouldn't be so much of an issue but "catch" doesn't work so the app has no choice but to crash
…On 1 Jun 2017, 4:15 AM +1000, Héctor Ramos ***@***.***>, wrote:
Reopened #13114.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Fully native apps do manage to not crash when performing the same functionally. I'll figure it out and submit a PR
…On 1 Jun 2017, 4:15 AM +1000, Héctor Ramos ***@***.***>, wrote:
Reopened #13114.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Yeah it’s not that unfortunately (though admittedly I did forget about that so I spent some time scratching my head). That correctly fails silently.
…On 1 Jun 2017, 6:07 AM +1000, Héctor Ramos ***@***.***>, wrote:
I found some references in the code to a change introduced in iOS 9 where the developer must whitelist schemes in Info.plist under the LSApplicationQueriesSchemes key. The expected behavior for opening a non-whitelisted scheme is for the operation to fail silently however.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
I have the issue with a
|
I have the same here. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. If you think this issue should definitely remain open, please let us know why. Thank you for your contributions. |
hey @lprhodes @drpiou did you guys figure out a work around for this?
After some research: according to Apple URL Scheme docs - sounds like this problem is rooted in apple, not a problem with RN |
Description
I set up linking to send an email using 'mailto:anyone@anywhere.com?subject=hi' as the URL. When I clicked the link it crashes... because the user has REMOVED the mail app!
Reproduction
Linking.openURL('mailto:anyone@anywhere.com?subject=hi').catch(err => console.error('An error occurred', err));
Solution
I'm not quite sure tbh.
Additional Information
The text was updated successfully, but these errors were encountered: