department-of-veterans-affairs / va.gov-team

Public resources for building on and in support of VA.gov. Visit complete Knowledge Hub:
https://depo-platform-documentation.scrollhelp.site/index.html
281 stars 198 forks source link

Full Accessibility & 508 Office Audit Benefits Delivery at Discharge #9081

Open luke-gcio opened 4 years ago

luke-gcio commented 4 years ago

Steps to complete the Full Accessibility & 508 Audit:

Artifacts - please provide the following artifacts when you request a full review

Accessibility Checks Completed — Self Check Stories:

  1. 8890 526 "Original Claims" and "All Claims"

  2. 9799 526 BDD

Test Cases

BDD flow:

Additional Notes

As part of this review, VSP will send your product to VA's 508 office for approval (as applicable)

jenstrickland commented 4 years ago

@luke-gcio - Has the team completed the accessibility test cases documentation? I recommend using the checklist method to certify that the team completed the required accessibility testing. Please let me know if I may be of help to get this audit ready to go.

luke-gcio commented 4 years ago

Spoke to VSP - with the understanding that BDD is part of Disability Claims, they are ok with combining this when we get to BDD.

jenstrickland commented 4 years ago

👍 Is this ticket a request for the full accessibility audit now?

luke-gcio commented 4 years ago

We have combined 526 and BDD so we need to do a full audit for both at the same time (they are tied at the hip). We are ready to begin! I will go make sure the test accounts for staging are correct.

jenstrickland commented 4 years ago

It's okay to do this as a two-in-one. Thanks!

jenstrickland commented 3 years ago

@luke-gcio @Mottie

Would y'all please add the URLs to access and the Users to test to the details above? Also, it would be helpful to document the test cases to go through to test BDD.

Mottie commented 3 years ago

BDD flow:

joshkimux commented 3 years ago

Not sure if a ticket has already been written up for the content here. Tagging @peggygannon for awareness!

Screen Shot 2021-01-14 at 11 01 08 AM
peggygannon commented 3 years ago

Hi, @joshkimux Thank you... This content above is outdated. I've added below a link to the content outline we had recommended to the team for the 526/BDD wizard work and implementation. Please let me know if you have any questions.

https://github.com/department-of-veterans-affairs/va.gov-team/blob/master/products/public-websites/how-to-apply-wizards/content-outlines/disability-form.md

cc @luke-gcio @johnhashva

joshkimux commented 3 years ago

Not an accessibility issue, but more of a usability one @Mottie @kevinstachura On the review page, if the user selects "no," they will lose all of their uploaded documents and will have to re-upload all of them if they re-select "yes"

https://user-images.githubusercontent.com/14154792/107548271-48ba7880-6b83-11eb-9601-57f5d4b529f1.mp4

joshkimux commented 3 years ago

Another usability issue here. Empty states do not have unique messaging: No-empty-state-disabilities

joshkimux commented 3 years ago

@1Copenut @luke-gcio I've tentatively completed this full a11y audit 🎉 Special thanks to @Mottie and @kevinstachura for answering all my questions and carrying the effort 🙇

I've noted this as tentative as additional issues may emerge while working on these tickets. Furthermore, I couldn't provide clear recommendations on some of the tickets as I think they deserve more conversation and thought. @1Copenut would love your review on these to see if they have appropriate severity levels and recommended solutions 🙏

Mottie commented 3 years ago

Hmm, that summary page shouldn't ever be blank. It'll contain any selected rated disability and all new disabilities. This may happen if you reload the page? If not, please let me know which user and if you made selections/added new disabilities. And if this happened after continuing a form, or you found it while stepping through the form in a single sitting. Thanks!

Mottie commented 3 years ago

That file upload problem you found... ouch... yeah that definitely needs to be fixed. I'll open a ticket.

Update: https://github.com/department-of-veterans-affairs/va.gov-team/issues/19806

joshkimux commented 3 years ago

@Mottie I'll check it again tomorrow! Fingers crossed it was just the browser being weird.