Skip to content
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

feat(router): Support linking to mailto:, and other common links with the <Link /> component and router API. #25486

Merged
merged 2 commits into from
Nov 20, 2023

Conversation

EvanBacon
Copy link
Contributor

Why

It is common to have links to compose emails and send messages.

How

If a URL isn't relative and matches a well-known list of URI schemes, then link externally. In the future we should change this logic so we first try matching internally, then match external if we get no results.

Test Plan

Added tests for the cases.

Checklist

…th the `<Link />` component and `router` API.
@expo-bot expo-bot added the bot: suggestions ExpoBot has some suggestions label Nov 20, 2023
Co-authored-by: Expo Bot <34669131+expo-bot@users.noreply.github.com>
@expo-bot expo-bot added bot: passed checks ExpoBot has nothing to complain about and removed bot: suggestions ExpoBot has some suggestions labels Nov 20, 2023
@EvanBacon EvanBacon merged commit dd2f9b7 into main Nov 20, 2023
7 of 8 checks passed
@EvanBacon EvanBacon deleted the @evanbacon/router/support-common-links branch November 20, 2023 22:56
onizam95 pushed a commit to onizam95/expo-av-drm that referenced this pull request Jan 15, 2024
…th the `<Link />` component and `router` API. (expo#25486)

# Why

It is common to have links to compose emails and send messages.

# How

If a URL isn't relative and matches a well-known list of URI schemes,
then link externally. In the future we should change this logic so we
first try matching internally, then match external if we get no results.

# Test Plan

Added tests for the cases.

# Checklist

<!--
Please check the appropriate items below if they apply to your diff.
This is required for changes to Expo modules.
-->

- [ ] Documentation is up to date to reflect these changes (eg:
https://docs.expo.dev and README.md).
- [ ] Conforms with the [Documentation Writing Style
Guide](https://github.com/expo/expo/blob/main/guides/Expo%20Documentation%20Writing%20Style%20Guide.md)
- [ ] This diff will work correctly for `npx expo prebuild` & EAS Build
(eg: updated a module plugin).

---------

Co-authored-by: Expo Bot <34669131+expo-bot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: fingerprint changed bot: passed checks ExpoBot has nothing to complain about
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants