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

API Updates for beta branch #1666

Merged
merged 7 commits into from
Jan 27, 2023
Merged

API Updates for beta branch #1666

merged 7 commits into from
Jan 27, 2023

Conversation

anniel-stripe
Copy link
Contributor

@anniel-stripe anniel-stripe commented Jan 26, 2023

Codegen for openapi v221.
cc @stripe/api-libraries

Changelog

  • Updated stable APIs to the latest version
  • Add support for billing_agreement_id on CheckoutSessionCreateParams.payment_method_options.paypal
  • Add support for list_transactions method on resource Tax.Transaction
  • Change type of QuoteUpdateParams.subscription_data_overrides from array(create_specs) to emptyStringable(array(update_specs))

* Separate stripe into worker and default entrypoints, needs tests for worker

* Fix test, remove unused import

* Actually require child_process

* build

* Convert tests to TypeScript, add browser field

* Move more utils functions to nodeUtils

* Use _utils defined on Stripe object

* Add error messages to createNodeHttpClient and createNodeCryptoProvider

* Move platform-specific utils to new PlatformFunctions classes, add tests

* Abstract getUname to platform functions, set _platformFunctions on Webhooks

* Make generic createCryptoProvider in Webhooks

* Revert "Make generic createCryptoProvider in Webhooks"

This reverts commit c8af4d5.

* Remove stale file in lib

* Create generic createHttpClient that creates FetchHttpClient by default in workers

* Move getUname to PlatformFunctions (now returns a promise), and misc reviewer comments

* Make generic createCryptoProvider in Webhooks

* Removing this alias

* Fix DefaultPlatformFunctions getUname, fix test

* Add getMockPlatformFunctions test util, prefix _createCryptoProvider
Copy link
Contributor

@pakrym-stripe pakrym-stripe left a comment

Choose a reason for hiding this comment

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

I think we lost DEFAULT_API_VERSION setting logic

@anniel-stripe anniel-stripe merged commit adf4cf3 into beta Jan 27, 2023
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.

2 participants