-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create Requesting Entity Page #2771
Comments
Moved to backlog. Just need to be pointed and this can be marked Ready, cc: @abroddrick. (The title does still say DRAFT. Can that be removed?) |
Removed draft, edited content, and removed myself as it appears refined. |
Based on @abroddrick last comment status of ticket, I am moving this ticket to "Ready". |
@CocoByte / @abroddrick See image below. Since this ticket is adding new city, org name, etc fields - do we have a ticket that'll update the csvs? |
@abroddrick / @CocoByte The AC "...on admin, hide fields when the user selects the "Other" option. Reference this annotation" links to an annotation that seems to refer to the form on the main page itself rather than Django admin. Regarding django admin specifically, it is difficult to do this without actually adding a dummy "other" record in the DB which I think we should avoid. I could of course inject an "other" option using javascript under certain conditions, but I think that might get overly engineered. |
An idea...
I've had to upgrade our form "section toggling magic" in my last PR...so maybe take a look at that first to see where I'm coming from. |
Issue description
The introduction of suborganizations means users need a way to specify whether their domain request belongs to the portfolio or a suborg. The solution for this is to create a "Requesting Entity" section in the domain request flow, which allows the user to make this specification.
This Requesting entity feature consists of the following parts;
Acceptance criteria
Additional context
The previous ticket #2760 added a placeholder requesting entity page that you can use here.
Links to other issues
The text was updated successfully, but these errors were encountered: