Page MenuHomePhabricator

Fix issue with "Publish/Notify" handling in repositories
ClosedPublic

Authored by epriestley on Sep 28 2015, 6:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 12, 9:07 PM
Unknown Object (File)
Thu, Apr 11, 7:30 AM
Unknown Object (File)
Tue, Apr 2, 9:36 PM
Unknown Object (File)
Mar 10 2024, 2:49 PM
Unknown Object (File)
Mar 10 2024, 2:14 PM
Unknown Object (File)
Mar 10 2024, 1:56 PM
Unknown Object (File)
Mar 10 2024, 12:28 PM
Unknown Object (File)
Mar 10 2024, 11:25 AM
Subscribers
None

Details

Summary

Fixes T8728. As far as I can tell, I simply got this wrong in D11826. This is not the proper name for the preference.

That change primarily focused on the "spammy junk during import" issue, and the code did get the importing flag right. It looks like my testing in D11827 focused on "during import" and just missed this case.

Test Plan

Grepped for disable-herald. Grepped for herald-disable.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Fix issue with "Publish/Notify" handling in repositories.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Sep 28 2015, 6:16 PM
This revision was automatically updated to reflect the committed changes.