Page MenuHomePhabricator

Allow the "Create Subtask" workflow to prompt for a subtype selection, and prepare for customizable options
ClosedPublic

Authored by epriestley on Dec 6 2018, 7:09 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 22, 9:20 PM
Unknown Object (File)
Fri, Nov 22, 12:11 AM
Unknown Object (File)
Thu, Nov 21, 8:10 PM
Unknown Object (File)
Mon, Nov 18, 9:44 AM
Unknown Object (File)
Wed, Nov 13, 4:11 PM
Unknown Object (File)
Tue, Nov 12, 8:58 AM
Unknown Object (File)
Sun, Nov 10, 3:34 PM
Unknown Object (File)
Wed, Nov 6, 4:09 PM
Subscribers
None

Details

Summary

Ref T13222. Ref T12588. See PHI683. Currently, "Create Subtask" always uses the first edit form that the user has access to for the same task subtype. (For example, if you "Create Subtask" from a "Bug", you get the first edit form for "Bugs".)

I didn't want to go too crazy with the initial subtype implementation, but it seems like we're generally on firm ground and it's working fairly well: user requests are for more flexibility in using the system as implemented, not changes to the system or confusion/difficulty with any of the tradeoffs. Thus, I'm generally comfortable continuing to build it out in the same direction. To improve flexibility, I want to make the options from "Create Subtask" more flexible/configurable.

I plan to let you specify that a given subtype (say, "Quest") prompts you with creation options for a set of other subtypes (say, "Objective"), or prompts you with a particular set of forms.

If we end up with a single option, we just go into the current flow (directly to the edit form). If we end up with more than one option, we prompt the user to choose between them.

This change is a first step toward this:

  • When building "Create Subtask", query for multiple forms.
  • The default behavior is now "prompt user to choose among create forms of the same subtype". Previously, it was "use the first edit form of the same subtype". This is a behavioral change.
  • The next change will make the selected forms configurable.
  • (I also plan to make the dialog itself less rough.)
Test Plan

Screen Shot 2018-12-06 at 10.57.08 AM.png (1×1 px, 155 KB)

Diff Detail

Repository
rP Phabricator
Branch
subtype2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21258
Build 28918: Run Core Tests
Build 28917: arc lint + arc unit