Page MenuHomePhabricator

Make Herald action modularization more aggressive
ClosedPublic

Authored by epriestley on Jul 17 2015, 5:31 PM.
Tags
None
Referenced Files
F15532766: D13646.diff
Wed, Apr 23, 5:13 PM
F15525214: D13646.id33276.diff
Mon, Apr 21, 12:51 PM
F15520394: D13646.id32992.diff
Sun, Apr 20, 6:06 AM
F15519060: D13646.id.diff
Sat, Apr 19, 7:38 PM
F15514730: D13646.diff
Fri, Apr 18, 4:58 AM
F15463775: D13646.diff
Wed, Apr 2, 3:12 AM
F15430051: D13646.id.diff
Mar 24 2025, 5:29 AM
F15425787: D13646.id32992.diff
Mar 23 2025, 6:04 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.

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
69

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.