Page MenuHomePhabricator

Show invoices on account information page
ClosedPublic

Authored by epriestley on Jan 31 2015, 4:09 PM.
Tags
None
Referenced Files
F14066047: D11593.diff
Tue, Nov 19, 7:54 AM
F14052685: D11593.diff
Fri, Nov 15, 9:42 AM
F14041104: D11593.diff
Mon, Nov 11, 4:00 PM
F14023374: D11593.diff
Thu, Nov 7, 1:18 AM
F14006681: D11593.diff
Mon, Oct 28, 3:12 PM
F13994569: D11593.id.diff
Wed, Oct 23, 7:06 AM
F13992953: D11593.diff
Tue, Oct 22, 7:47 PM
F13991978: D11593.diff
Tue, Oct 22, 1:38 PM
Subscribers
Tokens
"Love" token, awarded by johnny-bit.

Details

Summary

Ref T6881. This is basically just some UX.

Right now, if we invoice you, you can technically pay it but since we don't tell you about it and don't show it in the UI you'd have to guess the ID by manipulating the URI. We should probably be at least a little more aggressive about billing.

In the common case when we generate a cart/order, we don't show it to the user or merchant in Phortune until the user takes a payment action (basically, Phortune doesn't recognize the cart until you actually check out with it). In the current use case in Fund (and other reasonable use cases) an un-acted-upon cart hasn't been ordered yet, and is just a place for the application to store state as it hands off the workflow to Phortune.

Even if we had a real "Shop for physical goods" app, I think the same rule would apply -- the application itself would probably track and show your current cart, but it wouldn't make sense to put it into your order history in Phortune until you actually buy it.

Since invoices from subscriptions are essentially identical to not-yet-ordered-carts, that mean they also did not show up in the UI (although I think this is also desirable).

This change carves out a place for them:

  • Add an "invoices" section with unpaid invoices.
  • The UI shows that you have unpaid invoices.
  • Invoices have a slightly different rendering, inclduing an alluring "Pay Now" button.

Some considerations:

  • One thing I'm vaguely thinking about is the possibilty that users may be able to invoice one another directly, eventually. For example, we might invoice a contracting client.
  • Considering this, I thought about making these carts have a special status like STATUS_DUE, which replaces STATUS_READY, or a flag like isInvoice.
  • However, this approach was pretty involved and made the billing logic more complicated, so I backed off. The ultimate approach here puts more of the complexity into the display logic, which feels better to me.
  • We might need an isInvoice flag eventually, but subscriptionPHID is a reasonable stand-in for now.
  • The OrderTable serving double duty for rendering subscriptions feels a little muddy, but I think splitting it into two highly-redundant classes would be worse.
Test Plan

Screen_Shot_2015-01-31_at_7.53.10_AM.png (1×1 px, 240 KB)

Screen_Shot_2015-01-31_at_7.53.56_AM.png (1×1 px, 248 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable