Page MenuHomePhabricator
Differential D8501

Add CustomField to Diffusion and add a "branches" field
ClosedPublic
Actions

Authored by epriestley on Mar 12 2014, 4:36 PM.
Tags
None
Referenced Files
F15558548: D8501.id20177.diff
Tue, Apr 29, 3:44 AM
F15555407: D8501.id20169.diff
Mon, Apr 28, 11:14 AM
F15555260: D8501.id.diff
Mon, Apr 28, 10:34 AM
F15554601: D8501.id20168.diff
Mon, Apr 28, 7:43 AM
F15552765: D8501.diff
Sun, Apr 27, 10:26 PM
F15491095: D8501.id20177.diff
Fri, Apr 11, 11:32 PM
F15431604: D8501.id20169.diff
Mar 24 2025, 1:23 PM
F15424965: D8501.diff
Mar 23 2025, 1:23 AM
View All Files
Subscribers
aran
epriestley
zeeg

Details

Reviewers
btrahan
Maniphest Tasks
T4588: Add "branches" / "tags" information in Diffusion commit emails
Commits
Restricted Diffusion Commit
rP611f6707204c: Add CustomField to Diffusion and add a "branches" field
Summary

Ref T4588. Request from @zeeg. Adds a "BRANCHES" field to commit emails, so the branches the commit appears on are shown.

I've implemented this with CustomField, but in a very light way.

Test Plan

Used scripts/repository/reparse.php --herald to generate mail, got a BRANCHES section where applicable.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Diff 20168.Mar 12 2014, 4:36 PM
epriestley retitled this revision from to Add CustomField to Diffusion and add a "branches" field.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T4588: Add "branches" / "tags" information in Diffusion commit emails.
epriestley added a subscriber: zeeg.
Herald added subscribers: epriestley, aran.Mar 12 2014, 4:36 PM
epriestley updated this revision to Diff 20169.Mar 12 2014, 4:37 PM
  • Remove some dead code.
btrahan accepted this revision.Mar 12 2014, 6:15 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 12 2014, 6:15 PM
epriestley closed this revision.Mar 12 2014, 6:30 PM
epriestley updated this revision to Diff 20177.

Closed by commit rP611f6707204c (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
5 lines
applications/
repository/
customfield/
37 lines
7 lines
storage/
27 lines
worker/
17 lines
infrastructure/
customfield/
field/
18 lines

Diff 20177

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/repository/customfield/PhabricatorCommitBranchesField.php

Loading...
View Options

src/applications/repository/customfield/PhabricatorCommitCustomField.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepositoryCommit.php

Loading...
View Options

src/applications/repository/worker/PhabricatorRepositoryCommitHeraldWorker.php

Loading...
View Options

src/infrastructure/customfield/field/PhabricatorCustomField.php

Loading...
Phabricator · Built by Phacility