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

Revert DeepLinks in favor of regular links. #2312

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jpd236
Copy link
Contributor

@jpd236 jpd236 commented Oct 22, 2024

This component was originally added in 2017 to deep link to native Sheets/Docs apps when installed on mobile devices. On reasonably modern devices, regular HTTP links should be intercepted by native apps when present, so no special machinery is needed.

Fixes #1963
Fixes #2282

This component was originally added in 2017 to deep link to native
Sheets/Docs apps when installed on mobile devices. On reasonably modern
devices, regular HTTP links should be intercepted by native apps when
present, so no special machinery is needed.

Fixes deathandmayhem#1963
Fixes deathandmayhem#2282
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

On mobile, first tap on "Sheet" link doesn't do anything Mobile detection in DeepLink is broken
1 participant