Skip to content

Bug fixes for V1BankAccounts and V1Cards services #2098

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

Merged

Conversation

mbroshi-stripe
Copy link
Contributor

@mbroshi-stripe mbroshi-stripe commented Aug 1, 2025

Why?

In #2093, it was brought to our attention that there was a bug in the stripe.Client.V1BankAccounts service's Create method. It turns out there were actually a few bugs in both the V1BankAccounts and V1Cards services, which are non-standard Stripe services that we did not correctly port over to the new stripe.Client pattern.

What?

  • Adds custom parameter serialization to BankAccountCreateParams and CardCreateParams
  • Fixes Create and List methods in V1BankAccounts and V1Cards services.
  • Adds nil-checking to Retrieve, Update, and Delete methods in V1BankAccounts and V1Cards services.
  • Adds unit tests for both services

See Also

Changelog

  • Fixes bugs in Create and List methods in V1BankAccounts and V1Cards services in the stripe.Client pattern, which were previously returning errors on any valid inputs.

@mbroshi-stripe mbroshi-stripe requested a review from a team as a code owner August 1, 2025 20:38
@mbroshi-stripe mbroshi-stripe requested review from xavdid-stripe and removed request for a team August 1, 2025 20:38
@mbroshi-stripe mbroshi-stripe changed the title Mbroshi/run devsdk 1828/fix payment method services Bug fixes for V1BankAccounts and V1Cards services Aug 1, 2025
Copy link
Member

@xavdid-stripe xavdid-stripe left a comment

Choose a reason for hiding this comment

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

PR looks good!

Could you be a little more specific than "fixes bugs" in your changelog? Namely, we want to communicate to a reader if they would have been affected and any action they need to take

@mbroshi-stripe
Copy link
Contributor Author

Could you be a little more specific than "fixes bugs" in your changelog? Namely, we want to communicate to a reader if they would have been affected and any action they need to take

Fair point. In this case, these methods were basically broken, so any user attempting to use them would have received an error like that reported in #2093. Not sure how much gory detail to add there, but I will mention that they were previously broken altogether.

@mbroshi-stripe mbroshi-stripe merged commit 9a08760 into master Aug 3, 2025
11 checks passed
@mbroshi-stripe mbroshi-stripe deleted the mbroshi/RUN_DEVSDK-1828/fix-payment-method-services branch August 3, 2025 16:33
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