Page MenuHomePhabricator

When Herald changes object subscribers, always hide the feed story

Authored by epriestley on Sep 9 2019, 8:14 PM.



Fixes T8952. These feed stories are not interesting and tend to be generated as collateral damage when a non-story update is made to an old task and someone has a "subscribe me" Herald rule.

Also clean up some of the Herald field/condition indexing behavior slightly.

Test Plan

Wrote a "Subscribe X" herald rule, made a trivial update to a task. Before: low-value feed story; after: no feed story.

Diff Detail

rP Phabricator
Lint Not Applicable
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Sep 9 2019, 8:15 PM
epriestley requested review of this revision.
This revision was automatically updated to reflect the committed changes.