Page MenuHomePhabricator

Minor UX tweaks to Phortune autopay
ClosedPublic

Authored by chad on Aug 10 2017, 2:33 AM.
Tags
None
Referenced Files
F13226602: D18381.id44185.diff
Sun, May 19, 9:49 PM
F13210537: D18381.diff
Fri, May 17, 4:57 AM
F13194487: D18381.diff
Sun, May 12, 9:36 PM
F13177293: D18381.diff
Wed, May 8, 7:36 PM
F13175869: D18381.diff
Wed, May 8, 8:12 AM
Unknown Object (File)
Mon, May 6, 9:23 PM
Unknown Object (File)
Mon, Apr 29, 2:17 PM
Unknown Object (File)
Sat, Apr 27, 11:15 AM
Subscribers

Details

Summary

Fixes T12958. Adds a success message when card is added, also switches to use radio buttons for clarity. Updated redirect uri for deleting methods as well.

Test Plan

Add cards, remove cards.

image.png (1×2 px, 145 KB)

Diff Detail

Repository
rP Phabricator
Branch
phortune-methods (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 17973
Build 24134: Run Core Tests
Build 24133: arc lint + arc unit

Event Timeline

The concept of Autopay seems odd to me personally, at least it seems common in like insurance and other things, but not usually hosted billing from what I can find.

This revision is now accepted and ready to land.Aug 10 2017, 10:07 AM
This revision was automatically updated to reflect the committed changes.