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

Stripe: Check the correct status for completed checkout sessions. #1426

Merged
merged 1 commit into from
Nov 8, 2024

Conversation

dd32
Copy link
Member

@dd32 dd32 commented Nov 8, 2024

In #1256 the code 'succeeded' was used, instead of the documented 'complete'.

https://docs.stripe.com/api/checkout/sessions/object#checkout_session_object-status

This appears to have resulted in this code branch never having worked.

In #1256 the code 'succeeded' was used, instead of the documented 'complete'.

https://docs.stripe.com/api/checkout/sessions/object#checkout_session_object-status

This appears to have resulted in this code branch never having worked.
@dd32 dd32 requested a review from pkevan November 8, 2024 02:47
@dd32 dd32 merged commit 4ec9ce1 into production Nov 8, 2024
3 checks passed
@dd32 dd32 deleted the fix/stripe-timeouts branch November 8, 2024 03:27
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