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
Unknown Object (File)
Mon, Jun 8, 11:49 AM
Unknown Object (File)
Apr 8 2026, 11:01 AM
Unknown Object (File)
Mar 28 2026, 5:20 AM
Unknown Object (File)
Mar 15 2026, 1:55 PM
Unknown Object (File)
Mar 13 2026, 4:52 AM
Unknown Object (File)
Jan 10 2026, 9:25 AM
Unknown Object (File)
Dec 21 2025, 4:40 PM
Unknown Object (File)
Nov 9 2025, 10:48 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
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).