Page MenuHomePhabricator

Un-hide the "X added reviewers: ..." transactions in revision creation mail
ClosedPublic

Authored by epriestley on Sep 6 2017, 4:45 PM.
Tags
None
Referenced Files
F18729337: D18542.id.diff
Tue, Sep 30, 11:01 AM
F18721928: D18542.diff
Mon, Sep 29, 9:53 PM
F18716680: D18542.id44536.diff
Mon, Sep 29, 12:18 PM
F18705202: D18542.id44536.diff
Sun, Sep 28, 10:28 AM
F18222561: D18542.id44536.diff
Aug 19 2025, 9:17 PM
F18185574: D18542.id44541.diff
Aug 16 2025, 4:27 PM
F18185265: D18542.id44536.diff
Aug 16 2025, 3:37 PM
F18084399: D18542.id44536.diff
Aug 5 2025, 3:17 PM
Subscribers
None

Details

Summary

Fixes T12118. See PHI54. This adds a special case for the initial "reviewers" transactions, similar to the existing special case for "projects" transactions.

Although these transactions are redudnant in the web view since you can see the information clearly on the page, they're more reasonably useful in mail.

Test Plan

Screen Shot 2017-09-06 at 9.43.17 AM.png (167×367 px, 17 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable