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

Allow user to enter confirmation email and verify address before submission #334

Closed
AureliaKhorsand opened this issue Jun 10, 2022 · 6 comments
Assignees

Comments

@AureliaKhorsand
Copy link
Contributor

AureliaKhorsand commented Jun 10, 2022

Business Reason

As a user, I will be able to enter up to two email addresses to receive FEC confirmation of submission and verify that the committee address has or has not changed so that I can send the most updated information for FEC upon submission.

Acceptance Criteria

Given the user is on the "Confirm Information" page
Then they will be able to enter two Confirmation Emails, and the system will validate they are in the correct email format:

  1. Email # 1 - this should default to the email from Form 1, and the user has the option to change if needed. This is a required field.
  2. Email # 2 - this field defaults to blank, and is optional.

And they will be able to indicate if their address has changed or not, and this will always default to NO.
And if the user selects "YES", then the user will see their current address of the committee in editable textboxes, and have the ability to edit the address

  • STREET ADDRESS (A/N-34, required)
  • APARTMENT, SUITE, ETC. (A/N-34, not required)
  • CITY (A/N-30, required)
  • STATE/PROVINCE (required. Should be a dropdown) {note: will show written out state name and A/N-2 on back-end}
  • ZIP/POSTAL CODE (A/N-9, required)

And they can Save and Confirm, and go back and edit as needed before the report is submitted.

https://images.zenhubusercontent.com/306118986/7c61f9db-194f-4526-bf2b-34aa50dac37c/submit.mp4

Confirm Information - 1 Blank.png

QA Notes

To get to the confirmation page, edit a report and the report navigation in the left sidebar will appear. Select "SUBMIT YOUR REPORT" -> "Confirmation information"

DEV Notes

null

Design

null

@AureliaKhorsand AureliaKhorsand self-assigned this Jun 10, 2022
@AureliaKhorsand AureliaKhorsand changed the title Allow user to sign Form 3X report User must enter eFiling password to submit report to FEC Jun 21, 2022
@AureliaKhorsand AureliaKhorsand added external dependency UX/UI attention needed UX/UI tasks and/or redesign needed labels Jun 22, 2022
@AureliaKhorsand AureliaKhorsand changed the title User must enter eFiling password to submit report to FEC Allow user to enter eFiling password and confirmation emails when submitting report Jun 22, 2022
@AureliaKhorsand AureliaKhorsand changed the title Allow user to enter eFiling password and confirmation emails when submitting report Allow user to enter submission metadata when submitting report Jun 22, 2022
@MitchellTCG
Copy link
Contributor

@AureliaKhorsand Submit will be two steps. I'm not sure how you want to split the pages. We can discuss. Please review the prototype video.

@AureliaKhorsand AureliaKhorsand changed the title Allow user to enter submission metadata when submitting report Allow user to enter confirmation email and verify address before submmission Jul 5, 2022
@AureliaKhorsand AureliaKhorsand removed the UX/UI attention needed UX/UI tasks and/or redesign needed label Jul 5, 2022
@AureliaKhorsand
Copy link
Contributor Author

Refinement Meeting 7/6: BA action item - add requirements for verifying address to this ticket since it will need to be added into automated testing for this page.

@Elaine-Krauss-TCG Elaine-Krauss-TCG changed the title Allow user to enter confirmation email and verify address before submmission Allow user to enter confirmation email and verify address before submission Jul 11, 2022
@mjtravers
Copy link
Contributor

Some requests were made in code review. Sending ticket back to In Progress.

@mjtravers
Copy link
Contributor

Looks good for CR. Just need the unit test coverage increased over 80%. Sending back to In Progress for unit tests and one ANDI issue

@mjtravers
Copy link
Contributor

Passes CR, sending to QA

@WiseQA
Copy link

WiseQA commented Jul 25, 2022

QA review completed and verified Confirming of emails and address as denoted in Wireframes.

Test Case/Test Script attached showing ticket “PASS”.
https://app.zenhub.com/files/306118986/bd33bb7e-914d-4d0d-83d0-2f848e663a00/download

QA review completed. Moved to Stage Ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants