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

payment_intent.failed or payment_intent.payment_failed event type #609

Closed
ofavre opened this issue Mar 15, 2019 · 3 comments · Fixed by #645
Closed

payment_intent.failed or payment_intent.payment_failed event type #609

ofavre opened this issue Mar 15, 2019 · 3 comments · Fixed by #645

Comments

@ofavre
Copy link

ofavre commented Mar 15, 2019

The \Stripe\Event::PAYMENT_INTENT_FAILED constant is equal to 'payment_intent.failed', but the actual webhook event type I'm getting is 'payment_intent.payment_failed' as described in the docs.
I did not find information about it being renamed in the changelogs.
Is the event type still subject to changes as PaymentIntent is in beta or is it just a mistake that the old name was added in this library's code instead of the new one?

@ob-stripe
Copy link
Contributor

Hi @ofavre, thanks for reporting this. I'm fairly sure this is just a mistake and the name has always been payment_intent.payment_failed on the API's side. We'll fix this soon.

@ob-stripe
Copy link
Contributor

This is fixed in version 6.34.3. The \Stripe\Event::PAYMENT_INTENT_FAILED is gone and replaced with \Stripe\Event::PAYMENT_INTENT_PAYMENT_FAILED, which has the correct value.

@ob-stripe
Copy link
Contributor

(Sorry it took us so long to fix this!)

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 a pull request may close this issue.

2 participants