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.

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

Severity	Location	Code	Message
Advice	src/applications/audit/editor/PhabricatorAuditEditor.php:451	XHP16	TODO 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

Closed by commit rPcebbca9e08e4 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

audit/

editor/

PhabricatorAuditEditor.php

43 lines

Diff 24715

View Options

src/applications/audit/editor/PhabricatorAuditEditor.php

Add a "USERS" section to audit emails listing commit authors and committersClosedPublicActions