Page MenuHomePhabricator

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

Authored by epriestley on Jan 12 2017, 3:47 PM.
Tags
None
Referenced Files
F13314537: D17191.diff
Tue, Jun 11, 9:23 PM
F13309519: D17191.id41338.diff
Mon, Jun 10, 10:18 AM
F13304267: D17191.diff
Sat, Jun 8, 8:55 AM
F13293836: D17191.id41341.diff
Wed, Jun 5, 1:50 PM
F13293157: D17191.diff
Wed, Jun 5, 6:58 AM
F13276064: D17191.diff
Fri, May 31, 5:30 AM
F13262472: D17191.diff
Mon, May 27, 2:10 AM
F13245793: D17191.diff
Thu, May 23, 6:14 AM
Subscribers
None

Details

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

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

This revision is now accepted and ready to land.Jan 12 2017, 4:29 PM
This revision was automatically updated to reflect the committed changes.