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

Add a warning regarding opting out of the new architecture. #1816

Merged

Conversation

seanzhang-stripe
Copy link
Collaborator

@seanzhang-stripe seanzhang-stripe commented Jan 24, 2025

Summary

Add a warning regarding opting out of the new architecture.

Screenshot 2025-01-28 at 9 09 05 AM

Motivation

Testing

  • I tested this manually
  • I added automated tests

Documentation

Select one:

  • I have added relevant documentation for my changes.
  • This PR does not result in any developer-facing changes.

README.md Outdated
@@ -5,6 +5,9 @@

The Stripe React Native SDK allows you to build delightful payment experiences in your native Android and iOS apps using React Native. We provide powerful and customizable UI screens and elements that can be used out-of-the-box to collect your users' payment details.

> [!WARNING]
> This SDK is not fully compatible with React Native's [New Architecture](https://reactnative.dev/blog/2024/10/23/the-new-architecture-is-here). If you are using React Native 0.76 or above, please [opt-out](https://reactnative.dev/blog/2024/10/23/the-new-architecture-is-here#opt-out) from it.
Copy link
Collaborator

Choose a reason for hiding this comment

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

should we indicate that we intend to upgrade to the new architecture at some point? I worry people will see this and think that we have no intention to upgrade

@jaynewstrom-stripe jaynewstrom-stripe merged commit d33bd07 into stripe:master Jan 28, 2025
4 of 6 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.

4 participants