Page MenuHomePhabricator
Differential D10542

Provide more information in Herald Commit Email notifications - Add a list of merged commits to merge commits. - Show the author's name
AbandonedPublic
Actions

Authored by fabe on Sep 23 2014, 3:57 PM.
Tags
None
Referenced Files
F18816419: D10542.diff
Tue, Oct 21, 6:45 AM
F18790576: D10542.id.diff
Wed, Oct 15, 3:42 PM
F18766261: D10542.id25327.diff
Tue, Oct 7, 3:49 PM
F18758088: D10542.id31267.diff
Sun, Oct 5, 8:44 PM
F18727084: D10542.id27602.diff
Tue, Sep 30, 7:33 AM
F18651805: D10542.diff
Sep 21 2025, 5:33 AM
F18587290: D10542.id26272.diff
Sep 11 2025, 8:57 PM
F18585359: D10542.id31267.diff
Sep 11 2025, 3:13 PM
View All Files
Subscribers
epriestley
joshuaspence
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T8295: Show which commits were included in a merge commit in Herald emails
Summary

Add extra information about author and merges into herald commit emails

Test Plan

Create a herald email alert that hits a merge commit

Diff Detail

Repository
rP Phabricator
Branch
showMerges
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 6207
Build 6228: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

fabe updated this revision to Diff 25326.Sep 23 2014, 3:57 PM
fabe retitled this revision from to Provide more information in Herald Commit Email notifications - Add a list of merged commits to merge commits. - Show the author's name.
fabe updated this object.
fabe edited the test plan for this revision. (Show Details)
fabe added a reviewer: epriestley.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptSep 23 2014, 3:57 PM
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
Harbormaster completed remote builds in B2613: Diff 25326.Sep 23 2014, 3:58 PM
fabe updated this revision to Diff 25327.Sep 23 2014, 4:44 PM
fabe edited edge metadata.
  • only add the author info to the body if we cannot put it into the from tag
Harbormaster completed remote builds in B2614: Diff 25327.Sep 23 2014, 4:45 PM
fabe updated this revision to Diff 26272.Dec 6 2014, 1:10 PM

rebaseing

fabe updated this revision to Diff 27602.Jan 23 2015, 12:40 PM

rebasing

joshuaspence added a subscriber: joshuaspence.Feb 27 2015, 4:13 AM
fabe updated this revision to Diff 31267.May 22 2015, 7:36 AM

rebase

fabe added a task: T8295: Show which commits were included in a merge commit in Herald emails.May 22 2015, 7:44 AM
fabe abandoned this revision.May 25 2015, 10:31 AM

Revision Contents
Changeset List

PathSize
src/
applications/
audit/
editor/
49 lines

Diff 31267

View Options

src/applications/audit/editor/PhabricatorAuditEditor.php

Loading...
Phabricator · Built by Phacility