Page MenuHomePhabricator

Make Herald action modularization more aggressive
ClosedPublic

Authored by epriestley on Jul 17 2015, 5:31 PM.
Tags
None
Referenced Files
F18862665: D13646.id.diff
Nov 2 2025, 8:09 PM
F18855306: D13646.diff
Nov 1 2025, 4:49 AM
F18834435: D13646.diff
Oct 26 2025, 8:38 AM
F18811974: D13646.id33276.diff
Oct 20 2025, 6:13 AM
F18768247: D13646.diff
Oct 8 2025, 2:33 AM
F18743084: D13646.id.diff
Oct 2 2025, 10:19 PM
F18736051: D13646.diff
Oct 1 2025, 4:45 AM
F18727430: D13646.id32992.diff
Sep 30 2025, 7:56 AM

Details

Summary

Ref T8726. Herald actions are technically sort-of modular already, but make them more aggressively modular similar to HeraldField.

I plan to obsolete and replace HeraldCustomAction.

Test Plan

Saw actions in nice groups; created and ran a "Do Nothing" action. Transcripts are a bit rough for now.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Make Herald action modularization more aggressive.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
joshuaspence added inline comments.
src/applications/herald/action/HeraldAction.php
70

I wonder if we can do this in a more general way, see D13541 for am example of why.

Should be moot since it's cached behind PhutilClassMapQuery for the specific concern there.

btrahan edited edge metadata.
This revision is now accepted and ready to land.Jul 20 2015, 6:15 PM
This revision was automatically updated to reflect the committed changes.