Differential D20739

In Phortune, send order email to account external addresses
ClosedPublic
Actions

Authored by epriestley on Aug 23 2019, 8:42 PM.

Details

Reviewers: None
Maniphest Tasks: T8389: Allow arbitrary email addresses to be CC'd on Phacility billing emails
T13366: Update Phortune to work better with "enterprise" billing/accounts departments
Commits: rPa0a38797127f: In Phortune, send order email to account external addresses

Summary

Depends on D20738. Ref T13366. Fixes T8389. Now that the infrastructure is in place, actually send email to external addresses.

Test Plan

Used bin/phortune invoice to generate invoices and saw associated external accounts receive mail in bin/mail list-outbound.

Diff Detail

Repository

rP Phabricator

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Aug 23 2019, 8:42 PM

Harbormaster completed remote builds in B23306: Diff 49448.Aug 23 2019, 8:43 PM

epriestley requested review of this revision.Aug 23 2019, 8:43 PM

epriestley added a child revision: D20740: Give the Phortune external portal an order view.Aug 24 2019, 6:06 PM

This revision was not accepted when it landed; it landed in state Needs Review.Aug 26 2019, 2:48 PM

Closed by commit rPa0a38797127f: In Phortune, send order email to account external addresses (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rPa0a38797127f: In Phortune, send order email to account external addresses.

Revision Contents
Changeset List

		Path
M		src/__phutil_library_map__.php (2 lines)
M		src/applications/phortune/application/PhabricatorPhortuneApplication.php (3 lines)
M		src/applications/phortune/controller/external/PhortuneExternalController.php (25 lines)
A	M	src/applications/phortune/controller/external/PhortuneExternalOrderController.php (40 lines)
M		src/applications/phortune/controller/external/PhortuneExternalOverviewController.php (3 lines)
M		src/applications/phortune/editor/PhortuneCartEditor.php (97 lines)
M		src/applications/phortune/query/PhortuneAccountEmailQuery.php (13 lines)
M		src/applications/phortune/storage/PhortuneAccountEmail.php (8 lines)
M		src/applications/phortune/storage/PhortuneCart.php (4 lines)
M		src/applications/phortune/view/PhortuneOrderTableView.php (25 lines)
M		src/applications/phortune/worker/PhortuneSubscriptionWorker.php (1 line)
M		src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php (8 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	49448	fd2748a		Aug 23 2019, 8:42 PM	★	★
Diff 2	49452	4e13551	rPa0a38797127f091db7cdc3c782fc693206937183	Aug 26 2019, 2:48 PM	★	★

Status	Author	Revision
Closed	epriestley	D20740 Give the Phortune external portal an order view
Closed	epriestley	D20739 In Phortune, send order email to account external addresses
Closed	epriestley	D20738 Add credential rotation and statuses (disabled, unsubscribed) to Phortune external email
Closed	epriestley	D20737 Roughly support external/email user views of Phortune recipts and invoices

Diff 49452

View Options

src/__phutil_library_map__.php

View Options

src/applications/phortune/application/PhabricatorPhortuneApplication.php

View Options

src/applications/phortune/controller/external/PhortuneExternalController.php

View Options

src/applications/phortune/controller/external/PhortuneExternalOrderController.php

View Options

src/applications/phortune/controller/external/PhortuneExternalOverviewController.php

View Options

src/applications/phortune/editor/PhortuneCartEditor.php

View Options

src/applications/phortune/query/PhortuneAccountEmailQuery.php

View Options

src/applications/phortune/storage/PhortuneAccountEmail.php

View Options

src/applications/phortune/storage/PhortuneCart.php

View Options

src/applications/phortune/view/PhortuneOrderTableView.php

View Options

src/applications/phortune/worker/PhortuneSubscriptionWorker.php

View Options

src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

In Phortune, send order email to account external addressesClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 49452

src/__phutil_library_map__.php

src/applications/phortune/application/PhabricatorPhortuneApplication.php

src/applications/phortune/controller/external/PhortuneExternalController.php

src/applications/phortune/controller/external/PhortuneExternalOrderController.php

src/applications/phortune/controller/external/PhortuneExternalOverviewController.php

src/applications/phortune/editor/PhortuneCartEditor.php

src/applications/phortune/query/PhortuneAccountEmailQuery.php

src/applications/phortune/storage/PhortuneAccountEmail.php

src/applications/phortune/storage/PhortuneCart.php

src/applications/phortune/view/PhortuneOrderTableView.php

src/applications/phortune/worker/PhortuneSubscriptionWorker.php

src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

In Phortune, send order email to account external addresses
ClosedPublic
Actions

Revision Contents
Changeset List