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

10-7959c (Section 4): Add EOB file upload screen for prescriptions #83201

Closed marywang2 closed 3 months ago

marywang2 commented 4 months ago

As a user, I want to be able to upload supporting documents when filling out the Other Health Insurance Certification form, so that I can get the coverage I need.

Form: 10-7959c Section 4: Provider EOB for prescriptions Change/update: Add new screen for schedule of benefits file upload Expected: User should be able to upload supporting documents if they have no EOB Figma: https://www.figma.com/design/sSPoS7R9HE6ZlxRESRy2ms/10-7959c-Form?node-id=1156-74632&t=CmbjxN3AMniiaxbg-0

Acceptance Criteria

michaelclement commented 3 months ago

Work is done in this PR. Still waiting on:

michaelclement commented 3 months ago
Screenshots of the new process matching those provided in the assignment: s1 s2
Screenshot 2024-05-29 at 12.06.16.png Screenshot 2024-05-29 at 12.09.31.png
michaelclement commented 3 months ago

@marywang2 this has been merged and will be review-able on staging shortly. Thanks!

marywang2 commented 3 months ago

@michaelclement a few changes:

michaelclement commented 3 months ago
michaelclement commented 3 months ago

@marywang2 All the above work has been implemented in this PR. Please review new screenshots on the PR description on GitHub

Waiting on:

michaelclement commented 3 months ago

screenshot of breadcrumb which is in a different PR:

Screenshot 2024-05-31 at 11.27.04.png
michaelclement commented 3 months ago

@marywang2 Merged, should be reviewable on staging shortly. Thanks!

marywang2 commented 3 months ago

)@michaelclement same comments as the other ticket:

michaelclement commented 3 months ago

@marywang2 I believe the first two bullets are taken care of, but please let me know if not.

Screenshot 2024-06-03 at 09.45.57.png
michaelclement commented 3 months ago

Side note on the breadcrumb: I set it to just say "home", but "VA.gov" is automatically added to the front by the site (I'm seeing this behavior on other forms/apps within va.gov). I'm guessing it's a policy thing for clarity?

michaelclement commented 3 months ago

Latest work is here

michaelclement commented 3 months ago

@marywang2 Just an update:

This PR is just waiting on:

Update: this has been merged and will be visible on staging shortly.

marywang2 commented 3 months ago

fyi @jamiefiore