Page MenuHomePhabricator
Differential D15167

Implement subproject/milestone conflict resolution rules
ClosedPublic
Actions

Authored by epriestley on Feb 2 2016, 9:07 PM.
Tags
None
Referenced Files
F19813551: D15167.id36618.diff
Wed, Mar 4, 9:43 AM
F19797393: D15167.diff
Fri, Feb 27, 7:13 PM
F19707279: D15167.diff
Wed, Feb 11, 11:07 AM
F19550394: D15167.id36619.diff
Jan 27 2026, 9:37 PM
F19541591: D15167.id.diff
Jan 22 2026, 5:21 AM
F19468798: D15167.id36619.diff
Jan 4 2026, 10:23 AM
F19443771: D15167.id36618.diff
Dec 29 2025, 5:39 AM
F19293290: D15167.id.diff
Dec 23 2025, 10:19 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T10010: Implement Subprojects
Commits
Restricted Diffusion Commit
rP268a9ced78da: Implement subproject/milestone conflict resolution rules
Summary

Ref T10010. When you try to add "Sprint 35" to a task, remove "Sprint 34", etc. Briefly:

  • A task can't be in Sprint 3 and Sprint 4.
  • A task can't be in "A" and "A > B" (but "A > B" and "A > C" are fine).
  • When a user makes an edit which would violate one of these rules, preserve the last tag in each group of conflicts.
Test Plan
  • Added fairly comprehensive tests.
  • Added a bunch of different tags to things, saw them properly exclude conflicting tags.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 36618.Feb 2 2016, 9:07 PM
epriestley retitled this revision from to Implement subproject/milestone conflict resolution rules.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T10010: Implement Subprojects.
chad accepted this revision.Feb 2 2016, 9:10 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Feb 2 2016, 9:10 PM
epriestley added a comment.Feb 2 2016, 9:12 PM

One possible weird consequence of these rules is that these tags are OK:

  • (Stonework → Iteration IV) + (Stonework → Masonry)

I think that's fine for now. If it ultimately turns out to be confusing, we could make adding Stonework → Masonry remove Stonework → Milestone X.

Closed by commit rP268a9ced78da: Implement subproject/milestone conflict resolution rules (authored by epriestley, committed by epriestley). · Explain WhyFeb 2 2016, 9:12 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
project/
__tests__/
132 lines
transactions/
editor/
133 lines
docs/
user/
userguide/
33 lines

Diff 36619

View Options

src/applications/project/__tests__/PhabricatorProjectCoreTestCase.php

Loading...
View Options

src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

Loading...
View Options

src/docs/user/userguide/projects.diviner

Loading...
Phabricator · Built by Phacility