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

Update stripe-js for Custom Checkout types #479

Merged
merged 4 commits into from
Mar 8, 2024

Conversation

danwang-stripe
Copy link
Contributor

Summary & motivation

Update version of stripe-js to v3.0.7 to pull in new Custom Checkout types

API review

Copy this template or link to an API review issue.

Testing & documentation

package.json Outdated
@@ -92,15 +92,15 @@
"react-dom": "18.1.0",
"react-test-renderer": "^18.0.0",
"rimraf": "^2.6.2",
"rollup": "^1.27.0",
"rollup": "^4.12.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

@danwang-stripe I don't think we can upgrade rollup in this PR

@madhav-stripe
Copy link
Contributor

r? @fruchtose-stripe

@madhav-stripe
Copy link
Contributor

build output on current master:

-rw-r--r--  1 madhav  staff  44871 Mar  5 10:25 react-stripe.d.ts
-rw-r--r--  1 madhav  staff  44871 Mar  5 10:25 react-stripe.esm.d.ts
-rw-r--r--  1 madhav  staff  41327 Mar  5 10:25 react-stripe.esm.js
-rw-r--r--  1 madhav  staff  42399 Mar  5 10:25 react-stripe.js
-rw-r--r--  1 madhav  staff  44871 Mar  5 10:25 react-stripe.umd.d.ts
-rw-r--r--  1 madhav  staff  46976 Mar  5 10:25 react-stripe.umd.js
-rw-r--r--  1 madhav  staff  44871 Mar  5 10:25 react-stripe.umd.min.d.ts
-rw-r--r--  1 madhav  staff  16435 Mar  5 10:25 react-stripe.umd.min.js

@danwang-stripe
Copy link
Contributor Author

r? @madhav-stripe

Merged master in with 776705b

Thanks for helping with the dependency upgrades!

@danwang-stripe danwang-stripe merged commit f10b594 into master Mar 8, 2024
2 checks passed
@danwang-stripe danwang-stripe deleted the danwang/bump-stripe-js branch March 8, 2024 18:53
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.

None yet

2 participants