Page MenuHomePhabricator

Make Phortune payment methods transaction-oriented and always support "Add Payment Method"
ClosedPublic

Authored by epriestley on Aug 16 2019, 9:56 PM.

Details

Summary

Depends on D20718. Ref T13366. Ref T13367.

  • Phortune payment methods currently do not use transactions; update them.
  • Give them a proper view page with a transaction log.
  • Add an "Add Payment Method" button which always works.
  • Show which subscriptions a payment method is associated with.
  • Get rid of the "Active" status indicator since we now treat "disabled" as "removed", to align with user expectation/intent.
  • Swap out of some of the super weird div-form-button UI into the new "big, clickable" UI for choice dialogs among a small number of options on a single dimension.
Test Plan
  • As a mechant-authority and account-authority, created payment methods from carts, subscriptions, and accounts. Edited and viewed payment methods.

Diff Detail

Repository
rP Phabricator
Branch
merchant8
Lint
Lint OK
Unit
Unit Tests OK
Build Status
Buildable 23261
Build 31957: Run Core Tests
Build 31956: arc lint + arc unit