Page MenuHomePhabricator
Differential D13514

Modularize all remaining Phriction Herald fields
ClosedPublic
Actions

Authored by epriestley on Jul 1 2015, 3:31 PM.
Tags
None
Referenced Files
F18788882: D13514.id.diff
Wed, Oct 15, 6:54 AM
F18742979: D13514.id.diff
Thu, Oct 2, 9:57 PM
F18739724: D13514.diff
Wed, Oct 1, 10:38 PM
F18703550: D13514.id32786.diff
Sun, Sep 28, 4:23 AM
F18652946: D13514.id32674.diff
Sep 21 2025, 10:20 AM
F18645983: D13514.diff
Sep 19 2025, 8:59 AM
F18637354: D13514.id32786.diff
Sep 17 2025, 7:17 AM
F18627811: D13514.id32674.diff
Sep 16 2025, 4:51 AM
View All Files
Subscribers
eadler
epriestley
• GoogleLegacy

Details

Reviewers
btrahan
Maniphest Tasks
T8726: Modularize Herald fields and actions
Commits
Restricted Diffusion Commit
rP68de3dae9ab0: Modularize all remaining Phriction Herald fields
Summary

Ref T8726.

Test Plan
  • Created a rule using all the fields.
  • Updated documents, saw rule apply, reviewed transcript.
  • Migrated stuff.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 32674.Jul 1 2015, 3:31 PM
epriestley retitled this revision from to Modularize all remaining Phriction Herald fields.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T8726: Modularize Herald fields and actions.
Herald added a subscriber: epriestley. · View Herald TranscriptJul 1 2015, 3:31 PM
btrahan accepted this revision.Jul 1 2015, 5:30 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jul 1 2015, 5:30 PM
eadler added a subscriber: eadler.Jul 3 2015, 7:48 PM
Closed by commit rP68de3dae9ab0: Modularize all remaining Phriction Herald fields (authored by epriestley, committed by epriestley). · Explain WhyJul 6 2015, 8:17 PM
This revision was automatically updated to reflect the committed changes.
• GoogleLegacy added a subscriber: • GoogleLegacy.Sep 10 2018, 9:07 AM
This comment was removed by epriestley.
• GoogleLegacy added a comment.Sep 10 2018, 9:08 AM
This comment was removed by epriestley.

Revision Contents
Changeset List

Diff 32786

View Options

resources/sql/autopatches/20150701.herald.2.sql

Loading...
View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/herald/adapter/HeraldAdapter.php

Loading...
View Options

src/applications/maniphest/herald/HeraldManiphestTaskAdapter.php

Loading...
View Options

src/applications/pholio/herald/HeraldPholioMockAdapter.php

Loading...
View Options

src/applications/phriction/herald/PhrictionDocumentAuthorHeraldField.php

Loading...
View Options

src/applications/phriction/herald/PhrictionDocumentContentHeraldField.php

Loading...
View Options

src/applications/phriction/herald/PhrictionDocumentHeraldAdapter.php

Loading...
View Options

src/applications/phriction/herald/PhrictionDocumentHeraldField.php

Loading...
View Options

src/applications/phriction/herald/PhrictionDocumentPathHeraldField.php

Loading...
View Options

src/applications/phriction/herald/PhrictionDocumentTitleHeraldField.php

Loading...
Phabricator · Built by Phacility