Paths

Table of Contentst

Hide "X created Y, a subtask of P." feed stories again
ClosedPublic
Actions

Authored by epriestley on Jun 1 2017, 2:59 PM.

Tags

None

Referenced Files

	F15586121: D18058.id.diff
	Thu, May 8, 1:04 PM

	F15582449: D18058.diff
	Wed, May 7, 4:15 PM

	F15489212: D18058.id43434.diff
	Fri, Apr 11, 6:57 AM

	F15488925: D18058.diff
	Fri, Apr 11, 4:19 AM

	F15484765: D18058.id43435.diff
	Wed, Apr 9, 8:01 PM

	F15479155: D18058.diff
	Apr 8 2025, 5:21 AM

	F15467077: D18058.id.diff
	Apr 3 2025, 12:36 PM

	F15465206: D18058.diff
	Apr 2 2025, 4:04 PM

View All Files

Subscribers

None

Details

Reviewers

chad

Maniphest Tasks

T12787: Creating a subtask 'reopens' the new task

Commits

rP995c1503e7ef: Hide "X created Y, a subtask of P." feed stories again

Summary

Fixes T12787. Modular Transactions don't actually support shouldHideForFeed(). I'll add some discussion to the task.

Test Plan

Created a subtask, saw no more "X reopened Y, a subtask of P" feed story.

Diff Detail

Repository

rP Phabricator

Branch

sub1

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 17347
Build 23249: Run Core Tests
Build 23248: arc lint + arc unit

Event Timeline

epriestley created this revision.Jun 1 2017, 2:59 PM

Harbormaster completed remote builds in B17347: Diff 43434.Jun 1 2017, 3:00 PM

Thx

This revision is now accepted and ready to land.Jun 1 2017, 3:04 PM

Closed by commit rP995c1503e7ef: Hide "X created Y, a subtask of P." feed stories again (authored by epriestley, committed by epriestley). · Explain WhyJun 1 2017, 3:08 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in T12787: Creating a subtask 'reopens' the new task.Jun 1 2017, 3:14 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

maniphest/

storage/

ManiphestTransaction.php

16 lines

xaction/

ManiphestTaskUnblockTransaction.php

9 lines

Diff 43434

View Options

src/applications/maniphest/storage/ManiphestTransaction.php