Differential D17944

Clarify Subscription page language
ClosedPublic
Actions

Authored by chad on May 17 2017, 9:17 PM.

Details

Reviewers

epriestley

Maniphest Tasks

T12451: Finding your subscriptions / cards on Phacility is non-obvious

Commits

rP0e8f72a0d992: Clarify Subscription page language

Summary

This says "Edit Subscription" but the page only lets you update the payment method for autopay. I think this is confusing users. I tried to find the best language here, but suggest something else if you prefer.

Test Plan

Review language on sample subscription page.

Diff Detail

Repository

rP Phabricator

Branch

edit-subscription (branched from master)

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 17074
Build 22832: Run Core Tests
Build 22831: arc lint + arc unit

Event Timeline

chad created this revision.May 17 2017, 9:17 PM

Herald added a subscriber: Korvin. · View Herald TranscriptMay 17 2017, 9:17 PM

Harbormaster completed remote builds in B17074: Diff 43158.May 17 2017, 9:17 PM

Harbormaster completed remote builds in B17074: Diff 43158.

chad added a task: T12451: Finding your subscriptions / cards on Phacility is non-obvious.May 17 2017, 9:43 PM

epriestley accepted this revision.May 17 2017, 9:44 PM

This revision is now accepted and ready to land.May 17 2017, 9:44 PM

Maybe just "Manage Autopay" since that's the only thing in the UI.

Closed by commit rP0e8f72a0d992: Clarify Subscription page language (authored by chad, committed by chad). · Explain WhyMay 17 2017, 9:48 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

phortune/

controller/

subscription/

PhortuneSubscriptionViewController.php

4 lines

Diff 43158

View Options

src/applications/phortune/controller/subscription/PhortuneSubscriptionViewController.php

Clarify Subscription page languageClosedPublicActions