Page MenuHomePhabricator

Support basic ad-hoc invoices in Phortune
ClosedPublic

Authored by epriestley on Apr 20 2015, 3:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 26, 3:19 PM
Unknown Object (File)
Mon, Apr 22, 1:45 AM
Unknown Object (File)
Thu, Apr 11, 6:56 AM
Unknown Object (File)
Mar 17 2024, 1:15 AM
Unknown Object (File)
Feb 19 2024, 4:38 AM
Unknown Object (File)
Feb 19 2024, 4:38 AM
Unknown Object (File)
Feb 8 2024, 6:43 PM
Unknown Object (File)
Feb 8 2024, 4:26 PM
Subscribers

Details

Reviewers
btrahan
Commits
Restricted Diffusion Commit
rP0195e751c64d: Support basic ad-hoc invoices in Phortune
Summary

This allows a merchant to send a user an invoice for something arbitrary, like services rendered.

Two major missing parts:

  • These don't actually get marked as invoices. I'll fix that in the next diff, but it's not entirely trivial because subscriptionPHID is currently overloaded to also mean "is invoice".
  • We don't send email automatically. I don't plan to fix that for now, since all our invoicing needs are covered by personal email.
Test Plan

Merchants have a new "new invoice" option:

Screen_Shot_2015-04-20_at_8.34.29_AM.png (600×988 px, 37 KB)

This leads to selecting a user and account, and then you can generate the invoice (only one actual "purchase" / line item for the moment). You can add a longer-form remarkup description to contextualize the billable items:

Screen_Shot_2015-04-20_at_8.35.24_AM.png (1×2 px, 414 KB)

This sends the invoice and takes you to the merchant order overview screen:

Screen_Shot_2015-04-20_at_8.35.30_AM.png (1×2 px, 453 KB)

For now, you copy/paste that link into a nice personal enterprisey business-to-business email; the recipient sees this:

Screen_Shot_2015-04-20_at_8.35.51_AM.png (1×2 px, 484 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Support basic ad-hoc invoices in Phortune.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
btrahan added inline comments.
src/applications/phortune/controller/PhortuneCartViewController.php
52–53

i suggest removing the first "yet".

src/applications/phortune/controller/PhortuneMerchantInvoiceCreateController.php
129

*provide

This revision is now accepted and ready to land.Apr 20 2015, 4:56 PM
epriestley edited edge metadata.
  • Grammar/spelling fixes.
This revision was automatically updated to reflect the committed changes.