Page MenuHomePhabricator
Differential D18059

Remove non-operational `shouldHideFromFeed()` from ManiphestTaskPointsTransaction
ClosedPublic
Actions

Authored by epriestley on Jun 1 2017, 3:20 PM.
Tags
None
Referenced Files
F18094710: D18059.id.diff
Thu, Aug 7, 10:18 PM
F18052122: D18059.id43436.diff
Mon, Aug 4, 1:27 AM
F17900018: D18059.id.diff
Jul 29 2025, 8:52 AM
F17879491: D18059.diff
Jul 28 2025, 8:53 PM
F17829981: D18059.diff
Jul 26 2025, 10:34 AM
Unknown Object (File)
Jun 20 2025, 7:31 AM
Unknown Object (File)
Jun 17 2025, 3:12 PM
Unknown Object (File)
Jun 17 2025, 12:47 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T12787: Creating a subtask 'reopens' the new task
Commits
rPb69174a4c8f7: Remove non-operational `shouldHideFromFeed()` from…
Summary

See D18018. Ref T12787. This doesn't actually work; we started publishing these stories as a side effect of converting to ModularTransactions, then I fixed the rendering.

This mechanism has very few callsites and I suspect we may want to get rid of it (see T12787) so just keep publishing these stories for now.

Test Plan

Changed the point value of a task, saw a feed story both before and after the patch.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Jun 1 2017, 3:20 PM
Harbormaster completed remote builds in B17349: Diff 43436.Jun 1 2017, 3:21 PM
chad accepted this revision.Jun 1 2017, 4:09 PM
This revision is now accepted and ready to land.Jun 1 2017, 4:09 PM
Closed by commit rPb69174a4c8f7: Remove non-operational `shouldHideFromFeed()` from… (authored by epriestley, committed by epriestley). · Explain WhyJun 1 2017, 4:14 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
maniphest/
xaction/
4 lines

Diff 43437

View Options

src/applications/maniphest/xaction/ManiphestTaskPointsTransaction.php

Loading...
Phabricator · Built by Phacility