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
F15507698: D13514.id32674.diff
Tue, Apr 15, 10:00 PM
F15457879: D13514.id32786.diff
Sun, Mar 30, 7:45 PM
F15433566: D13514.diff
Mon, Mar 24, 11:46 PM
F15423104: D13514.id32786.diff
Mar 22 2025, 12:57 PM
F15413647: D13514.id32786.diff
Mar 19 2025, 8:19 PM
F15405625: D13514.id.diff
Mar 18 2025, 11:43 AM
F15393641: D13514.id.diff
Mar 15 2025, 10:56 PM
F15391524: D13514.id32674.diff
Mar 15 2025, 10:21 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