Page MenuHomePhabricator
Differential D17191

Restore "[Action]" mail subject lines to Differential/Diffusion
ClosedPublic
Actions

Authored by epriestley on Jan 12 2017, 3:47 PM.
Tags
None
Referenced Files
F15502439: D17191.id41341.diff
Mon, Apr 14, 12:45 AM
F15456728: D17191.diff
Sun, Mar 30, 11:44 AM
F15452996: D17191.id41338.diff
Sat, Mar 29, 8:38 AM
F15441586: D17191.id41338.diff
Wed, Mar 26, 6:51 PM
F15438141: D17191.diff
Tue, Mar 25, 11:36 PM
F15437237: D17191.diff
Tue, Mar 25, 6:29 PM
F15389299: D17191.diff
Mar 15 2025, 5:05 AM
Unknown Object (File)
Mar 3 2025, 1:43 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T10978: Modernize Audit
T11114: Move Differential to EditEngine
Commits
rP7d3d0224071a: Restore "[Action]" mail subject lines to Differential/Diffusion
Summary

Ref T11114. Ref T10978. These hadn't made it over to EditEngine yet.

Test Plan
  • Took various actions on revisions and commits.
  • Used bin/mail show-outbound --id ... to examine the "Vary Subject", saw it properly generate "[Accepted]", "[Resigned]", etc.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 12 2017, 3:47 PM
Harbormaster completed remote builds in B15226: Diff 41338.Jan 12 2017, 3:48 PM
epriestley added a comment.Jan 12 2017, 3:57 PM

I intentionally made some of these a little more concise ("Requested A Review Of" -> "Requested Review") since they were a little wordy/inconsistent.

chad accepted this revision.Jan 12 2017, 4:29 PM
This revision is now accepted and ready to land.Jan 12 2017, 4:29 PM
Closed by commit rP7d3d0224071a: Restore "[Action]" mail subject lines to Differential/Diffusion (authored by epriestley, committed by epriestley). · Explain WhyJan 12 2017, 7:44 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 41341

View Options

src/applications/differential/xaction/DifferentialRevisionAbandonTransaction.php

Loading...
View Options

src/applications/differential/xaction/DifferentialRevisionAcceptTransaction.php

Loading...
View Options

src/applications/differential/xaction/DifferentialRevisionActionTransaction.php

Loading...
View Options

src/applications/differential/xaction/DifferentialRevisionCloseTransaction.php

Loading...
View Options

src/applications/differential/xaction/DifferentialRevisionCommandeerTransaction.php

Loading...
View Options

src/applications/differential/xaction/DifferentialRevisionPlanChangesTransaction.php

Loading...
View Options

src/applications/differential/xaction/DifferentialRevisionReclaimTransaction.php

Loading...
View Options

src/applications/differential/xaction/DifferentialRevisionRejectTransaction.php

Loading...
View Options

src/applications/differential/xaction/DifferentialRevisionReopenTransaction.php

Loading...
View Options

src/applications/differential/xaction/DifferentialRevisionRequestReviewTransaction.php

Loading...
View Options

src/applications/differential/xaction/DifferentialRevisionResignTransaction.php

Loading...
View Options

src/applications/diffusion/xaction/DiffusionCommitAcceptTransaction.php

Loading...
View Options

src/applications/diffusion/xaction/DiffusionCommitConcernTransaction.php

Loading...
View Options

src/applications/diffusion/xaction/DiffusionCommitResignTransaction.php

Loading...
View Options

src/applications/transactions/storage/PhabricatorModularTransaction.php

Loading...
View Options

src/applications/transactions/storage/PhabricatorModularTransactionType.php

Loading...
Phabricator · Built by Phacility