See downstream https://phabricator.wikimedia.org/T166321. These got dropped in refactoring, or maybe never existed.
Details
Details
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
T12787 shed some light on this: previously, we used shouldHideFromFeed() to hide these stories completely.
ManiphestTaskPointsTransaction faithfully ported this over, but it doesn't currently work in ModularTransactions, so the previously hidden stories became visible.
I could go either way on continuing to publish these. I'm just going to leave them for the moment (and remove the inoperative shouldHideForFeed()) but we could completely hide them again in the future if they turn out to be too noisy.