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
F14066023: D8509.diff
Tue, Nov 19, 7:48 AM
F14053372: D8509.diff
Fri, Nov 15, 3:09 PM
F14038899: D8509.diff
Mon, Nov 11, 2:40 AM
F14024084: D8509.diff
Thu, Nov 7, 5:41 AM
F13979453: D8509.diff
Oct 19 2024, 4:39 AM
F13965470: D8509.diff
Oct 16 2024, 2:18 AM
Unknown Object (File)
Oct 9 2024, 3:38 PM
Unknown Object (File)
Oct 2 2024, 3:29 PM
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
Branch
disherald
Lint
Lint Passed
Unit
Tests Passed

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).