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

New release: v9.0.1 #122

Merged
merged 1 commit into from
Nov 26, 2021
Merged

New release: v9.0.1 #122

merged 1 commit into from
Nov 26, 2021

Conversation

Gnito
Copy link
Contributor

@Gnito Gnito commented Nov 26, 2021

Bug fix for default payment method usage that caused 409 with Stripe message:
"You cannot provide a new payment method to a PaymentIntent when it has a status of requires_capture. You should either capture the remaining funds, or cancel this PaymentIntent and try again with a new PaymentIntent."

Essentially, before confirming PaymentIntent, we now check if the PaymentIntent has already been confirmed (which might be done for default payment method).

Note: this should also remove the earlier warning that confirming already confirmed PaymentIntent is unnecessary.

(The bug was introduced after fixing another bug just before release...)

Changelog entries:

  • [fix] Fixes issue with default payment card that was created by PR 111. Essentially, before
    confirming PaymentIntent, we now check if the PaymentIntent has already been confirmed.
    #121
  • [fix] StripePaymentForm: add missing optional chaining: defaultPaymentMethod?.id
    #120
  • [fix] Temporarily disallow Node v17, since it causes issues with dependencies.
    #119

@Gnito Gnito merged commit 24083da into main Nov 26, 2021
@Gnito Gnito deleted the v9.0.1 branch November 26, 2021 12:50
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.

1 participant