Skip to content
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

Dev: Allow a domain requestor to tell us if they're working with a person from CISA regions #1895

Closed
8 tasks
vickyszuchin opened this issue Mar 11, 2024 · 3 comments · Fixed by #2013
Closed
8 tasks
Assignees
Labels
carryover Carryover from a previous sprint dev

Comments

@vickyszuchin
Copy link

vickyszuchin commented Mar 11, 2024

Issue description

Knowing that a requestor is working with staff from CISA regions can be helpful. We should ask a question about it in the request form. Follow the designs laid out in this Figma page (more detailed AC below)

Acceptance criteria

  • Refer to content google document as source of truth
  • On the Anything else? page, change page title to "Additional details"
  • Add These questions are required (*). under the page title
  • Add a new question asking if the user is working with a CISA regional rep (Figma link) Follow the google doc for content and Figma for design and placement
  • Move the Anything else question below the CISA regional rep question. This question is turning into a radio button instead of an optional open field. (Figma designs) Follow the google doc for content and Figma for design and placement
  • In Django admin expose these fields according to Figma designs
  • For behavior of the conditional fields, follow the logic used in the other employees page.
  • For errors, follow the errors laid out here and here

Additional context

No response

Links to other issues

Related to: #1891 (design ticket)

@vickyszuchin
Copy link
Author

"Block" tag added: depended on design ticket (#1891)

@vickyszuchin vickyszuchin added the carryover Carryover from a previous sprint label Mar 20, 2024
@vickyszuchin
Copy link
Author

vickyszuchin commented Mar 20, 2024

Moved out of sprint 41because of dependency on design ticket. Added "carryover" and added to sprint 42. cc: @abroddrick

@gabydisarli
Copy link
Contributor

Can we make a small tweak this error message?
Screenshot 2024-04-22 at 12 55 43 PM
Instead of "your" email, can we say Enter the email of your representative in the required format, like name@example.com.

@vickyszuchin vickyszuchin added PO and removed PO labels Apr 25, 2024
zandercymatics added a commit that referenced this issue Apr 25, 2024
…domain-req

Issue #1895: Allow a domain requestor to tell us if they're working with a person from CISA regions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
carryover Carryover from a previous sprint dev
Projects
Status: ✅ Done
5 participants