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
F19100820: D10542.diff
Fri, Dec 5, 12:50 AM
F19055407: D10542.id26272.diff
Fri, Nov 28, 5:26 PM
F18999763: D10542.id25327.diff
Thu, Nov 20, 9:55 PM
F18872037: D10542.id.diff
Nov 5 2025, 2:01 AM
F18850497: D10542.diff
Oct 30 2025, 5:25 PM
F18828841: D10542.id25326.diff
Oct 24 2025, 6:28 PM
F18821216: D10542.id.diff
Oct 22 2025, 8:34 PM
F18816419: D10542.diff
Oct 21 2025, 6:45 AM
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 4011
Build 4024: [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/
48 lines

Diff 27602

View Options

src/applications/audit/editor/PhabricatorAuditEditor.php

Loading...
Phabricator · Built by Phacility