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

docs: Add Deep Links guide #2581

Merged
merged 7 commits into from
Mar 19, 2020
Merged

docs: Add Deep Links guide #2581

merged 7 commits into from
Mar 19, 2020

Conversation

dotNetkow
Copy link
Contributor

Needed to implement this for a personal app, so turned it into a Capacitor guide.

Used my app for screenshots, video, etc examples - could switch to something more generic but thought it would be good to use a real example (instead of "yourwebsite.com" everywhere).

Still need to add React code, but wanted early feedback.

@mlynch
Copy link
Contributor

mlynch commented Mar 15, 2020

Nice! I think the link on the side should say "Deeplinking" which will be more obvious/identifiable by most people

@andysousa
Copy link
Contributor

nice, I am going to try this... noticed this am that I hadn't updated the config since moving to Capacitor and I would love to get rid of the Cordova plugin

@dotNetkow dotNetkow marked this pull request as ready for review March 18, 2020 18:53
@jcesarmobile jcesarmobile changed the title New guide: Deep links docs: Add Deep Links guide Mar 19, 2020
Copy link
Member

@jcesarmobile jcesarmobile left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a few comments.

I find confusing that we say deep links support web platform when App plugin doesn't have appUrlOpen event on web platform.
While is true that regular links work on web, deep links are related to apps, not to web.
Maybe reword in a way that it says that deep links will fallback to regular links on web or something like that, but don't say deep links are supported.

site/docs-md/guides/deep-links.md Outdated Show resolved Hide resolved
site/docs-md/guides/deep-links.md Outdated Show resolved Hide resolved
site/docs-md/guides/deep-links.md Show resolved Hide resolved
site/docs-md/guides/deep-links.md Outdated Show resolved Hide resolved
@dotNetkow dotNetkow merged commit b9e25f3 into master Mar 19, 2020
@dotNetkow dotNetkow deleted the docs-deeplinks branch March 19, 2020 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants