Skip to content

Add deferred deep linking section#1490

Merged
satya164 merged 3 commits into
react-navigation:mainfrom
BrtqKr:@brtqkr/deferred-deep-links-docs-update
May 14, 2026
Merged

Add deferred deep linking section#1490
satya164 merged 3 commits into
react-navigation:mainfrom
BrtqKr:@brtqkr/deferred-deep-links-docs-update

Conversation

@BrtqKr
Copy link
Copy Markdown
Contributor

@BrtqKr BrtqKr commented May 13, 2026

No description provided.

@netlify
Copy link
Copy Markdown

netlify Bot commented May 13, 2026

Deploy Preview for react-navigation-docs failed.

Name Link
🔨 Latest commit 0f80245
🔍 Latest deploy log https://app.netlify.com/projects/react-navigation-docs/deploys/6a05dc633855e00007590f79

Copy link
Copy Markdown
Member

@satya164 satya164 left a comment

Choose a reason for hiding this comment

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

Thanks for the PR.

I'd remove changes from 5.x and 6.x since you're probably not testing with them, and we aren't actively maintaining these versions.

Before adding the links, I'd also request to change your examples to use getInitialURL + subscribe pattern instead of using the current pattern using useEffect and ref https://github.com/software-mansion-labs/react-native-detour/blob/30666c7c18fea8995550231b388a1aece275122b/examples/react-navigation/src/App.tsx#L62-L74, as this pattern doesn't integrate with React Navigation. So many of React Navigation's features are not supported (e.g., automatic redirection to the correct screen after auth, validation, automatic deep links and type inference in static config, etc.). It also requires a lot of boilerplate since the user needs to handle mappings themselves.

Here is an example of getInitialURL + subscribe https://reactnavigation.org/docs/deep-linking/#integrating-with-other-tools. Happy to answer any questions (you can ask @kkafar to add you to our discord if you want to chat directly).

@BrtqKr
Copy link
Copy Markdown
Contributor Author

BrtqKr commented May 14, 2026

@kkafar mentioned that last week, appreciate the feedback! We've already assigned one of our devs to work on that, so I didn't want to bother you 😅 – the PR should be merged soon. Regarding the docs – removed v5 and v6.

Copy link
Copy Markdown
Member

@satya164 satya164 left a comment

Choose a reason for hiding this comment

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

Thank you

@satya164 satya164 merged commit cef48d4 into react-navigation:main May 14, 2026
4 of 5 checks passed
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.

2 participants