Page MenuHomePhabricator
Differential D10266

Add a "USERS" section to audit emails listing commit authors and committers
ClosedPublic
Actions

Authored by epriestley on Aug 14 2014, 3:49 PM.
Tags
None
Referenced Files
F18744895: D10266.id24720.diff
Fri, Oct 3, 4:32 AM
F18732043: D10266.id.diff
Tue, Sep 30, 3:59 PM
F18712094: D10266.id.diff
Mon, Sep 29, 5:16 AM
F18630428: D10266.id.diff
Tue, Sep 16, 11:39 AM
F18619896: D10266.id24720.diff
Mon, Sep 15, 3:40 AM
F18619505: D10266.diff
Mon, Sep 15, 2:40 AM
F18616864: D10266.id24715.diff
Sun, Sep 14, 7:17 PM
F18420254: D10266.id.diff
Aug 30 2025, 4:04 PM
View All Files
Subscribers
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T5872: Include commit author context in audit emails
Commits
Restricted Diffusion Commit
rPcebbca9e08e4: Add a "USERS" section to audit emails listing commit authors and committers
Summary

Fixes T5872. This won't show up in the initial email until T4896 is further along.

Test Plan
RECIPIENTS
  discoball (Disco Ball)

BODY
epriestley added a comment.

ffkn

USERS
  epriestley (Author)

COMMIT
  http://local.aphront.com:8080/rPOEMS165b6c54f487

Diff Detail

Repository
rP Phabricator
Branch
users
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/audit/editor/PhabricatorAuditEditor.php:451XHP16TODO Comment
Unit
No Test Coverage
Build Status
Buildable 2226
Build 2230: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 24715.Aug 14 2014, 3:49 PM
epriestley retitled this revision from to Add a "USERS" section to audit emails listing commit authors and committers.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T5872: Include commit author context in audit emails.
Herald added a subscriber: epriestley. · View Herald TranscriptAug 14 2014, 3:49 PM
Harbormaster completed remote builds in B2226: Diff 24715.Aug 14 2014, 3:50 PM
btrahan accepted this revision.Aug 14 2014, 6:49 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Aug 14 2014, 6:49 PM
epriestley closed this revision.Aug 14 2014, 7:14 PM
epriestley updated this revision to Diff 24720.

Closed by commit rPcebbca9e08e4 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
audit/
editor/
43 lines

Diff 24715

View Options

src/applications/audit/editor/PhabricatorAuditEditor.php

Loading...
Phabricator · Built by Phacility