Page MenuHomePhabricator

Disable Herald more aggressively when it's turned off for a repository
ClosedPublic

Authored by epriestley on Mar 12 2014, 10:41 PM.
Tags
None
Referenced Files
F15420997: D8509.id20189.diff
Fri, Mar 21, 7:59 PM
F15405863: D8509.id20200.diff
Tue, Mar 18, 11:58 AM
F15354427: D8509.diff
Tue, Mar 11, 3:57 AM
Unknown Object (File)
Sun, Mar 2, 3:23 AM
Unknown Object (File)
Feb 21 2025, 3:32 AM
Unknown Object (File)
Feb 17 2025, 1:36 PM
Unknown Object (File)
Feb 9 2025, 11:54 AM
Unknown Object (File)
Feb 9 2025, 11:54 AM
Subscribers
Tokens
"Love" token, awarded by dctrwatson.

Details

Summary

Currently, disabling Herald only disables feed, notifications and email. Historically, audits didn't really create external effects so it made sense for Herald to only partially disable itself.

With the advent of Harbormaster/Build Plans, it makes more sense for Herald to just stop doing anything. When this option is disabled, stop all audit/build/publish/feed/email actions for the repository.

Test Plan

Ran scripts/repository/reparse.php --herald, etc.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Disable Herald more aggressively when it's turned off for a repository.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: btrahan, dctrwatson.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 13 2014, 12:56 AM
epriestley updated this revision to Diff 20200.

Closed by commit rPd27cd5fb9902 (authored by @epriestley).