Page MenuHomePhabricator
Differential D8902

Restrict marking task duplicate to "merge duplicates" workflow
ClosedPublic
Actions

Authored by lpriestley on Apr 29 2014, 6:29 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 13, 3:19 AM
Unknown Object (File)
Fri, Dec 13, 12:06 AM
Unknown Object (File)
Fri, Dec 6, 12:20 PM
Unknown Object (File)
Wed, Dec 4, 3:10 PM
Unknown Object (File)
Tue, Nov 26, 7:50 PM
Unknown Object (File)
Tue, Nov 26, 5:23 PM
Unknown Object (File)
Nov 18 2024, 10:03 PM
Unknown Object (File)
Nov 16 2024, 4:25 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T4819: "Duplicate" status should not appear in Maniphest Edit UI unless a task is already closed as a duplicate
Commits
Restricted Diffusion Commit
rP1d5731b1ae3c: Restrict marking task duplicate to "merge duplicates" workflow
Summary

Fixes T4819, remove status "duplicate" from dropdown in edit task unless task is already in duplicate status

Test Plan

Edit task, not in duplicate status, verify dropdown does not have "duplicate" option. Edit task already in "duplicate" status, verify that dropdown shows "duplicate" status option.

Diff Detail

Repository
rP Phabricator
Branch
removedupstatus
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 131
Build 131: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

lpriestley updated this revision to Diff 21127.Apr 29 2014, 6:29 PM
lpriestley retitled this revision from to Restrict marking task duplicate to "merge duplicates" workflow.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
lpriestley added a task: T4819: "Duplicate" status should not appear in Maniphest Edit UI unless a task is already closed as a duplicate.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptApr 29 2014, 6:29 PM
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
Harbormaster completed remote builds in B131: Diff 21127.Apr 29 2014, 6:30 PM
epriestley accepted this revision.Apr 29 2014, 6:41 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Apr 29 2014, 6:41 PM
epriestley closed this revision.Apr 29 2014, 6:41 PM
epriestley updated this revision to Diff 21128.

Closed by commit rP1d5731b1ae3c (authored by @lkassianik, committed by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
maniphest/
controller/
10 lines

Diff 21127

View Options

src/applications/maniphest/controller/ManiphestTaskEditController.php

Loading...
Phabricator · Built by Phacility