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
F13248692: D18542.id.diff
Fri, May 24, 3:56 AM
F13236235: D18542.diff
Tue, May 21, 8:51 AM
F13228710: D18542.id44541.diff
Mon, May 20, 11:29 AM
F13224025: D18542.id44536.diff
Sun, May 19, 5:58 AM
Unknown Object (File)
Apr 29 2024, 7:47 PM
Unknown Object (File)
Apr 28 2024, 2:41 PM
Unknown Object (File)
Apr 27 2024, 11:42 PM
Unknown Object (File)
Apr 26 2024, 2:36 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
Branch
diffm1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18319
Build 24654: Run Core Tests
Build 24653: arc lint + arc unit