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
- Branch
- task1
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 17250 Build 23101: Run Core Tests Build 23100: arc lint + arc unit
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.