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

Mailing address edit subtask page, update field validation and cancel button #176

Closed
1 of 8 tasks
Tracked by #124
adamwhitlock1 opened this issue Oct 22, 2024 · 5 comments
Closed
1 of 8 tasks
Tracked by #124
Assignees
Labels
Added [Pattern] Update their prefilled information Authenticated Experience Design Pattern
Milestone

Comments

@adamwhitlock1
Copy link
Collaborator

adamwhitlock1 commented Oct 22, 2024

User Story

As an engineer I want the buttons to behave correctly when validation is incorrect or the cancel button

Background

  • make sure field validation is fired before save occurs
  • cancel button needs to go somewhere, most likely back to the previous page (the review page)

ACs

  • buttons work as expected

Review needed by

  • VA Product Owner
  • Product Manager
  • UX Research Team
  • Engineering Team

Definition of Done

  • Documentation has been updated, if applicable.
  • Acceptance Criteria in related issue are met.
  • Reviewers have approved.
@adamwhitlock1
Copy link
Collaborator Author

We have updated the prototype here: https://noxious-monster-gjxjrxqrrgfqvw-3001.app.github.dev/mock-form-ae-design-patterns/2/task-orange/introduction?loggedIn=false

It now includes the correct mailing address update screen that can be accessed by clicking the edit button on the review screen. Once it is viewed, you can remove the zip code or remove any other required field and attempt to save the form to view form validation.

@msbtterswrth
Copy link
Collaborator

msbtterswrth commented Oct 30, 2024

Looks like when you click the continue button on the contact info screen it takes you back to the personal info step, is that right @adamwhitlock1? This comment might apply to the other open task for this flow.

The editing does work as expected though.

@adamwhitlock1
Copy link
Collaborator Author

ah yes @msbtterswrth that issue should be fixed in my next PR, which is about ready for review. I'll ping you on that one when it is merged and available on this codespace, but there is some funkiness with the routing for the form system and it can result in navigating backwards if things aren't just right. Thanks

@msbtterswrth msbtterswrth changed the title Mailing address edit subtask page, update field validation and cancel buttoin Mailing address edit subtask page, update field validation and cancel button Oct 31, 2024
@msbtterswrth
Copy link
Collaborator

Sounds good, I'll close this one out when that comes through <3

@adamwhitlock1
Copy link
Collaborator Author

The newest updates should be available now on https://noxious-monster-gjxjrxqrrgfqvw-3001.app.github.dev/mock-form-ae-design-patterns/2

I just spun it up so it should be live for 4 hours, but if you want we can also go over things on Monday to make sure its all validated.

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Added [Pattern] Update their prefilled information Authenticated Experience Design Pattern
Projects
None yet
Development

No branches or pull requests

3 participants