KEMBAR78
[breaking] Update types for Checkout SDK by pololi-stripe · Pull Request #803 · stripe/stripe-js · GitHub
Skip to content

Conversation

@pololi-stripe
Copy link
Contributor

@pololi-stripe pololi-stripe commented Sep 3, 2025

@pololi-stripe pololi-stripe force-pushed the pololi/add-clover-custom-checkout-sdk-types branch 3 times, most recently from dbd131f to c1425a5 Compare September 3, 2025 21:40
@pololi-stripe pololi-stripe changed the title Update types for Checkout SDK [breaking] Update types for Checkout SDK Sep 3, 2025
@pololi-stripe pololi-stripe force-pushed the pololi/add-clover-custom-checkout-sdk-types branch from c1425a5 to 4fa8512 Compare September 3, 2025 21:47
* Contact [Stripe support](https://support.stripe.com/) for more information.
*/
initCheckout(options: StripeCheckoutOptions): Promise<StripeCheckout>;
initCheckout(options: StripeCheckoutOptions): StripeCheckout;
Copy link
Contributor

Choose a reason for hiding this comment

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

While we're here, can we remove the beta access notice 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed 0718c41


export interface StripeCheckout {
/* Custom Checkout methods */
type LoadActionsError = {message: string; code: null};
Copy link
Contributor

Choose a reason for hiding this comment

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

Should code be code: string | null?

Copy link
Contributor Author

@pololi-stripe pololi-stripe Sep 4, 2025

Choose a reason for hiding this comment

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

No, I think the idea is to be explicit about the error.code here.
Similar to other

type UpdateTaxIdInfoError =
| {message: string; code: 'invalidTaxId'}
| AnyBuyerError;

Right now, we don't have a concrete code and only has null
We can add specific ones when we introduce them in the future.

@pololi-stripe pololi-stripe merged commit 1a679ae into master Sep 9, 2025
2 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.

2 participants