Page MenuHomePhabricator

Handle Phortune charge failures cleanly
ClosedPublic

Authored by epriestley on Oct 8 2014, 10:33 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 15, 11:30 AM
Unknown Object (File)
Sun, Mar 10, 1:07 PM
Unknown Object (File)
Sun, Mar 10, 10:50 AM
Unknown Object (File)
Feb 20 2024, 9:14 AM
Unknown Object (File)
Feb 7 2024, 1:13 PM
Unknown Object (File)
Feb 7 2024, 1:13 PM
Unknown Object (File)
Feb 7 2024, 1:13 PM
Unknown Object (File)
Feb 7 2024, 12:57 PM
Subscribers

Details

Summary

Ref T2787. Currently, we kill a cart and dead-end the workflow on a charge failure.

Instead, fail the charge and reset the cart so the user can try using a valid payment instrument like a normal checkout workflow would.

Some shakiness/smoothing on WePay for the moment; PayPal is still made up since we don't have a "Hold" state yet.

Test Plan

Screen_Shot_2014-10-08_at_3.28.16_PM.png (1×1 px, 186 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Handle Phortune charge failures cleanly.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Oct 8 2014, 10:49 PM
This revision was automatically updated to reflect the committed changes.