[BD-4660] Update push notifications to include more guidance on deep links for React Native#12403
Open
[BD-4660] Update push notifications to include more guidance on deep links for React Native#12403
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds migration-focused guidance to the React Native push notifications docs to help readers troubleshoot deep link handling after upgrades.
Changes:
- Adds an Important alert calling out deep link validation after upgrading/migrating React Native push dependencies.
- Adds a new troubleshooting section for cases where push-notification deep links stop opening, including iOS closed-state handling pointers and support escalation guidance.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Contributor
christorbraze
left a comment
There was a problem hiding this comment.
lgtm, but you'll want to get a review from the SDKs team for accuracy
radixdev
approved these changes
Mar 6, 2026
atom-evens
approved these changes
Mar 6, 2026
Contributor
atom-evens
left a comment
There was a problem hiding this comment.
approved, with a few minor edits
|
|
||
| {% alert important %} | ||
| If you're migrating an existing React Native push integration, re-test deep linking after you upgrade the Braze SDK, React Native, Expo, or related libraries. Confirm that: | ||
| - React Native [Linking](https://reactnative.dev/docs/linking) is still configured and handling your deep link URLs. |
Contributor
There was a problem hiding this comment.
Suggested change
| - React Native [Linking](https://reactnative.dev/docs/linking) is still configured and handling your deep link URLs. | |
| - [React Native Linking](https://reactnative.dev/docs/linking) is still configured and handling your deep link URLs. |
|
|
||
| {% endtab %} | ||
| {% tab iOS Native %} | ||
| #### Step 3.1: Store the push notification payload on app launch |
Contributor
There was a problem hiding this comment.
Suggested change
| #### Step 3.1: Store the push notification payload on app launch {#step-3-1} |
| {% alert important %} | ||
| If you're migrating an existing React Native push integration, re-test deep linking after you upgrade the Braze SDK, React Native, Expo, or related libraries. Confirm that: | ||
| - React Native [Linking](https://reactnative.dev/docs/linking) is still configured and handling your deep link URLs. | ||
| - Your iOS initial push payload handling (see **Step 3.1: Store the push notification payload on app launch**) is implemented and still called on app launch. |
Contributor
There was a problem hiding this comment.
Suggested change
| - Your iOS initial push payload handling (see **Step 3.1: Store the push notification payload on app launch**) is implemented and still called on app launch. | |
| - Your iOS initial push payload handling (see [Step 3.1: Store the push notification payload on app launch](#step-3-1)) is implemented and still called on app launch. |
hokstuff
approved these changes
Mar 6, 2026
Contributor
hokstuff
left a comment
There was a problem hiding this comment.
LGTM after existing suggestions
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why are you making this change? (required)
The content introduced by this PR provides users with additional guidance on deep links for React Native. It also includes guidance on how to reach out to support if issues persist.
Related PRs, issues, or features (optional)
Feature release date (optional)
Contributor checklist
Submitting for review
If your PR meets the above requirements, select Ready for review, then add a reviewer:
@braze-inc/docs-teamin a comment below.braze-inc/docs-teaminstead.Thanks for contributing! We look forward to reading your work.