Upgrade to React 19

[cbala-stripe]

What happened?

We want to upgrade to React 19, but it's a more complex upgrade. This issue tracks the upgrade.

Working branch:
master...react-19

The branch pulls from #623 and:

1. Removed the import * as React changes to reduce scope
2. Rebased the branch and fixed some issues
3. Adds a required node version

There are some currently failing tests, and I'd also like to further investigate some of the test changes because we are dropping some assertions. We'll also need to test in real integrations and determine if this is a breaking change.

Thanks for being patient with this process—we still have some work to do here and are still aware that this is a change we need to make.

Environment

No response

Reproduction

No response

[cbala-stripe]

It looks like this upgrade may only affect our internal tools and doesn't affect packages that depend on this one. Either way, we should do this upgrade.

#623 (comment)

@jake-brandt can you confirm that you were able to safely upgrade to React 19?

[jake-brandt]

It looks like this upgrade may only affect our internal tools and doesn't affect packages that depend on this one. Either way, we should do this upgrade.

#623 (comment)

@jake-brandt can you confirm that you were able to safely upgrade to React 19?

@cbala-stripe you have impeccable timing, it's actively going through regression right now after being pushed down by a few recent priorities. I'll ask our QA engineer I was partnering with to see if it's been completed.

My largest concern, not being entirely close to the Stripe integration, was if this library created any automatic webhook endpoints for Stripe, and if so, that the routes used the new promise-based args for props and headers now. I can't see those working just by using a version override the way I did. I don't believe this library introduces any routes, however.

I will get back to you this afternoon.

[jake-brandt]

@cbala-stripe let me clarify - we're still using your mainline (forced Stripe into our R15 projects via version overrides), not our fork. Testing of #623 was limited to existing unit tests (modified as necessary to match any R15 changes).

[jake-brandt]

@cbala-stripe we are running v2.7.1 of @stripe/react-stripe-js as none of the newer major versions had updated to R19. On this version, we've just successfully run regression on:

• Adding cards
• Paying at checkout
• Monthly payments

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[mduchev]

Not stale

[Ghostavio]

What's the progress on this?

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.