Page MenuHomePhabricator

Don't show donors error messages when error occurs after successful payment
Closed, ResolvedPublic1 Estimated Story Points

Description

Sometimes a donor's payment succeeds, but we crash trying to send the donation message to the queue. We should never make the donor think they need to try again if we've actually charged them. Distinguish between pre- and post-payment errors and don't send donors to the fail page if we might have taken their money.

Event Timeline

Ejegg raised the priority of this task from to High.
Ejegg updated the task description. (Show Details)
Ejegg subscribed.

per @K4-713 in standup, let's send donors to TY page and just make noise internally

Ejegg renamed this task from Give donors better message when error occurs after successful payment to Don't show donors error messages when error occurs after successful payment.Dec 3 2015, 6:44 PM
Ejegg updated the task description. (Show Details)
Ejegg set Security to None.
Ejegg edited a custom field.

Change 256737 had a related patch set uploaded (by Ejegg):
Catch error queueing final status message

https://gerrit.wikimedia.org/r/256737

Change 256737 merged by jenkins-bot:
Catch error queueing final status message

https://gerrit.wikimedia.org/r/256737

Re-opening this because we're getting a lot of session timeouts lately: T149139

For Paypal classic, AstroPay, and Adyen, the payment is already going through by the time the donor gets to the result switcher. Many of these donors with session timeouts go on to make duplicate donations.

Instead of always showing the fail page on dead sessions with those processors, we should always show the thank you page.

Change 318015 had a related patch set uploaded (by Ejegg):
WIP: show TY page on dead session if potentially paid

https://gerrit.wikimedia.org/r/318015

Change 318015 merged by jenkins-bot:
Show TY page on dead session if potentially paid

https://gerrit.wikimedia.org/r/318015