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
F14762574: D8509.diff
Thu, Jan 23, 6:49 AM
F14759459: D8509.diff
Wed, Jan 22, 12:33 PM
Unknown Object (File)
Sat, Jan 18, 2:32 AM
Unknown Object (File)
Fri, Jan 17, 11:06 PM
Unknown Object (File)
Thu, Jan 16, 12:15 AM
Unknown Object (File)
Sat, Dec 28, 6:53 PM
Unknown Object (File)
Dec 13 2024, 9:39 PM
Unknown Object (File)
Dec 11 2024, 9:15 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
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).