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

Remove field validation on Military Service Number #90673

Closed
3 tasks
aplatt-coforma opened this issue Aug 13, 2024 · 6 comments
Closed
3 tasks

Remove field validation on Military Service Number #90673

aplatt-coforma opened this issue Aug 13, 2024 · 6 comments
Assignees
Labels
engineering Engineering topics frontend pension Project related work for VA Pension (527EZ)

Comments

@aplatt-coforma
Copy link
Contributor

aplatt-coforma commented Aug 13, 2024

Issue Description

With #88432, we are now pre-filling military history for those veterans who have that info available in their profile. One issue we encountered was when pre filling service number, Veterans who enrolled after 1971 have their SSN as their service number, so that's what we're using for them. The front-end validation for that field is trigged since it doesn't match the expected service number format.

VBA input:

Feedback from our VBA Stakeholder (Derek Uttley) is that we should remove format validation from this field. Any data provided in this field could be useful downstream and will save the Veteran some keystrokes, the VBA team will take whatever the system provides in this field.


Tasks

  • Remove the front end field format validation for the Military Service Number so the error message does not display regardless of what is entered or prefilled.

Acceptance Criteria

  • Field accepts all inputs regardless of character or number combos
  • User is able to get to the next step on the form regardless of what they put into the Military Service Number field
@matt4su
Copy link
Contributor

matt4su commented Aug 20, 2024

Perhaps unrelated to Service Number change which tested fine, I did see instances where the error messaging associated with the "Date Initially entered active duty" and "Final release date looked repeated and misaligned. Steps to recreate: On step 2 of 7 (Military history) in the pension app populate all required fields on the page, except leave blank the "day" or "year" value" of the Date initially entered active duty or Final release date from active duty. Then click Continue. Note the "Please enter a complete date" verbiage is repeated and the second instance of the verbiage pushes down the input box in a way I would not expect. Tested in Chrome Version 127.0.6533.120 (Official Build) (x86_64). See screenshot. Screenshot 2024-08-20 at 1.53.13 PM.png

@TaiWilkin
Copy link

@matt4su I'm seeing it for other date fields, so it's definitely not related to the service number change. We should make a ticket to look into it
Screenshot 2024-08-20 at 3 12 26 PM

@matt4su
Copy link
Contributor

matt4su commented Aug 20, 2024

@TaiWilkin it appears this issue is in another team's backlog. department-of-veterans-affairs/vets-design-system-documentation#3113. Once they address would we have to do anything additional to benefit from the change (e.g a pension app specific ticket to implement?)

@TaiWilkin
Copy link

@matt4su A fix to the component will likely fix our issue as well without additional work on our end, but we should check after the fix is released

@matt4su
Copy link
Contributor

matt4su commented Aug 20, 2024

Will do, thanks @TaiWilkin

@aplatt-coforma
Copy link
Contributor Author

Closing this ticket since Matt verified that the original issue is fixed.

Well test the issue noted in department-of-veterans-affairs/vets-design-system-documentation#3113 once it's ready on staging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
engineering Engineering topics frontend pension Project related work for VA Pension (527EZ)
Projects
None yet
Development

No branches or pull requests

3 participants