stable
Clone or download
Go to a pane other than pv2 (like overview or cardwall) on a given milestone Use the +New dropdown menu to add a new backlog item. => since there is no modal on this pane, you are redirected to the tracker in order to create the artifact. Fill the form and click submit => once the artifact created, you are redirected to the previous pane, so the chance to loose the context is decreased. Note: the behavior is kept if the user choose "submit and continue" to add multiple backlog items to a milestone. (Other panes like taskboard or testplan will be handled in a separate commit to not clutter the review) For the record, we cannot use the already existing mechanism "link-artifact-id" (in combination with the peculiar "immediate" parameter) since it is used for legacy overlay and does not manage well the display in case of form error at artifact submission. Therefore we have to introduce an AD specific parameter. Part of story #16206: « + New » button, better contextualisation Change-Id: Id711280f05308a18c03c82575af793f53641632c
Modified Files
| Name | ||||
|---|---|---|---|---|
| M | plugins/agiledashboard/include/AgileDashboard/Artifact/EventRedirectAfterArtifactCreationOrUpdateHandler.php | +5 | −7 | Go to diff View file |
| M | plugins/agiledashboard/include/AgileDashboard/Artifact/RedirectParameterInjector.php | +4 | −0 | Go to diff View file |
| M | plugins/agiledashboard/include/AgileDashboard/Milestone/HeaderOptionsProvider.php | +16 | −8 | Go to diff View file |
| M | plugins/agiledashboard/include/Planning/ArtifactLinker.class.php | +76 | −29 | Go to diff View file |
| M | plugins/agiledashboard/tests/unit/AgileDashboard/Artifact/EventRedirectAfterArtifactCreationOrUpdateHandlerTest.php | +50 | −6 | Go to diff View file |
| M | plugins/agiledashboard/tests/unit/AgileDashboard/Artifact/RedirectParameterInjectorTest.php | +17 | −0 | Go to diff View file |
| M | plugins/agiledashboard/tests/unit/AgileDashboard/Milestone/HeaderOptionsProviderTest.php | +39 | −8 | Go to diff View file |
| M | plugins/agiledashboard/tests/unit/AgileDashboard/Planning/ArtifactLinkerTest.php | +38 | −9 | Go to diff View file |
| M | plugins/tracker/include/Tracker/NewDropdown/TrackerLinksInNewDropdownCollector.php | +4 | −2 | Go to diff View file |
| M | plugins/tracker/include/Tracker/NewDropdown/TrackerNewDropdownLinkPresenterBuilder.php | +17 | −1 | Go to diff View file |
| M | plugins/tracker/include/Tracker/Tracker.class.php | +17 | −4 | Go to diff View file |
| M | plugins/tracker/tests/unit/Tracker/NewDropdown/TrackerLinksInNewDropdownCollectorTest.php | +7 | −1 | Go to diff View file |
| M | plugins/tracker/tests/unit/Tracker/NewDropdown/TrackerNewDropdownLinkPresenterBuilderTest.php | +45 | −13 | Go to diff View file |
| M | plugins/tracker/tests/unit/Tracker/TrackerTest.php | +10 | −0 | Go to diff View file |