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
Unknown Object (File)
Sun, Apr 14, 6:09 AM
Unknown Object (File)
Sun, Apr 14, 1:33 AM
Unknown Object (File)
Thu, Apr 11, 7:57 AM
Unknown Object (File)
Sat, Apr 6, 1:33 AM
Unknown Object (File)
Tue, Apr 2, 4:46 AM
Unknown Object (File)
Fri, Mar 29, 1:17 PM
Unknown Object (File)
Thu, Mar 28, 2:08 PM
Unknown Object (File)
Feb 7 2024, 3:26 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