Skip to content

[BD-4660] Update push notifications to include more guidance on deep links for React Native#12403

Open
zairro wants to merge 3 commits intodevelopfrom
zrko-react-native-deeplinks
Open

[BD-4660] Update push notifications to include more guidance on deep links for React Native#12403
zairro wants to merge 3 commits intodevelopfrom
zrko-react-native-deeplinks

Conversation

@zairro
Copy link
Contributor

@zairro zairro commented Mar 2, 2026

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)

  • BD-4660

Feature release date (optional)

  • N/A

Contributor checklist

  • I confirm that my PR meets the following:
    • I signed the Contribution License Agreement (CLA).
    • My style and voice follow the Braze Style Guide.
    • My content contains correct spelling and grammar.
    • All links are working correctly.
    • If I renamed or moved a file or directory, I set up URL redirects for each file.
    • If I updated or replaced an image, I did not remove the original image file from the repository. (For more information, see Updating an image).
    • If my PR is related to a paid SKU, third party, SMS, AI, or privacy, I have received written approval from Braze Legal.

Submitting for review

If your PR meets the above requirements, select Ready for review, then add a reviewer:

  • Non-Braze contributors: Tag @braze-inc/docs-team in a comment below.
  • Braze employees: Have any relevant subject matter experts (like engineers or product managers) review your work first, then add the tech writer assigned to your specific vertical. If you're not sure which tech writer to add, you can add braze-inc/docs-team instead.

Thanks for contributing! We look forward to reading your work.

Copilot AI review requested due to automatic review settings March 2, 2026 19:33
@zairro zairro changed the title Update push notifications to include more guidance on deep links for React Native [BD-4660] Update push notifications to include more guidance on deep links for React Native Mar 2, 2026
@zairro zairro requested a review from a team March 2, 2026 19:35
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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.

Copy link
Contributor

@christorbraze christorbraze left a comment

Choose a reason for hiding this comment

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

lgtm, but you'll want to get a review from the SDKs team for accuracy

@atom-evens atom-evens requested a review from a team March 5, 2026 21:39
Copy link
Contributor

@atom-evens atom-evens left a comment

Choose a reason for hiding this comment

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

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.
Copy link
Contributor

Choose a reason for hiding this comment

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

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
Copy link
Contributor

Choose a reason for hiding this comment

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

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.
Copy link
Contributor

Choose a reason for hiding this comment

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

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.

Copy link
Contributor

@hokstuff hokstuff left a comment

Choose a reason for hiding this comment

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

LGTM after existing suggestions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants