Page MenuHomePhabricator

Support Herald rules for Herald rules
ClosedPublic

Authored by epriestley on Apr 23 2018, 9:07 PM.
Tags
None
Referenced Files
F19105929: D19400.id.diff
Sat, Dec 6, 4:18 PM
F19104669: D19400.diff
Fri, Dec 5, 3:11 PM
F19067865: D19400.diff
Sun, Nov 30, 11:05 AM
F19046571: D19400.diff
Thu, Nov 27, 10:54 AM
F18866369: D19400.diff
Nov 3 2025, 4:48 PM
F18836507: D19400.diff
Oct 27 2025, 1:47 AM
F18814345: D19400.diff
Oct 20 2025, 11:02 PM
F18789161: D19400.diff
Oct 15 2025, 8:18 AM
Subscribers
None

Details

Summary

Depends on D19399. Ref T13130. This adds basic support for writing Herald rules against Herald rules. See T13130 for a lot more detail.

This needs a bit more work to be useful: for example, there's no way to specify the rule type or subject, so you can't say "notify me when global rules are edited" or "notify me when Maniphest rules are edited". I'll add some fields for that in followup changes to actually solve the original use case.

Test Plan
  • Wrote Herald rules against Herald rules.
  • Ran them by editing rules and in the test console.
  • Verified they sent some mail with bin/mail list-outbound.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Looks good except for cruft.

src/applications/herald/adapter/HeraldRuleAdapter.php
26

"a another"

src/applications/herald/controller/HeraldRuleController.php
378 ↗(On Diff #46404)

Leftover logging.

This revision is now accepted and ready to land.Apr 23 2018, 9:45 PM
  • Fix "a another".
  • Remove stray "phlog()".
This revision was automatically updated to reflect the committed changes.