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

Failed Payments via Stripe #2

Open
woodwardmatt opened this issue May 29, 2017 · 3 comments
Open

Failed Payments via Stripe #2

woodwardmatt opened this issue May 29, 2017 · 3 comments

Comments

@woodwardmatt
Copy link

Hi there,

I'm just trying to piece the "Failed Payment" process together and spotted this code on Line 271 of paid-memberships-pro/services/stripe-webhook.php:

	`elseif($pmpro_stripe_event->type` == "charge.failed")

Having looked at the data sent back by Stripe (for me) as part of the webhook, I'm just wondering if this should this be

	`elseif($pmpro_stripe_event->type` == "invoice.payment_failed")

(Or maybe cater for both?)...

	`elseif($pmpro_stripe_event->type` == "charge.failed" || $pmpro_stripe_event->type` == "invoice.payment_failed")

Otherwise the failed payment (where an invoice is being raised / paid) would never trigger the "pmpro_subscription_payment_failed" action within this plugin for PMP to cancel the membership?

Have I missed something? :) TYIA :)

@StevieDC
Copy link

Hi,
I'm just about to use this plugin and also use Stripe. Did you establish whether that change is ok/required? Thanks.

@woodwardmatt
Copy link
Author

Hey @StevieDC sorry for the slow response. I've just done a quick sanity check, and I don't believe we use this plugin for this purpose now.

We currently use the failed payment settings through Stripe, and have the subscription cancelled from there if all retries fail. This still occurs via PMPRO and works fine :)

Hope that helps!

@DBAD24
Copy link

DBAD24 commented Jul 4, 2024

Hey @StevieDC sorry for the slow response. I've just done a quick sanity check, and I don't believe we use this plugin for this purpose now.

We currently use the failed payment settings through Stripe, and have the subscription cancelled from there if all retries fail. This still occurs via PMPRO and works fine :)

Hope that helps!

Stripe are saying its our site cancelling the members after one failed attempt. Our Stripe settings are to try 3 times.

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

No branches or pull requests

3 participants