search

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
282 stars 201 forks source link

COE | View uploaded documents #52781

Open saderagsdale opened 1 year ago

saderagsdale commented 1 year ago

Value Statement

As a Veteran I want to see the documents I submitted with my application So that I can quickly and easily recall what I've shared.

Background Context

LGY application processors sometimes need to request additional documentation from Veterans. It's helpful for Veterans to see what they've already shared to minimize friction in the application process.

Acceptance Criteria

Designs and Build Notes

Out of scope

Tasks

saderagsdale commented 1 year ago

@uxgary Hey Gary - can you post a link to the draft design for review ahead of backlog refinement tomorrow?

uxgary commented 1 year ago

@saderagsdale Here's a link what I have in draft so far. https://sketch.com/s/b3237bfe-9e5c-4b5d-a958-66632d9a7fe3

saderagsdale commented 1 year ago

@uxgary review existing patterns on VA.gov, schedule design collab reivew.

saderagsdale commented 1 year ago

@uxgary will bring this up in design office hours.

saderagsdale commented 1 year ago

Sade to find ticket template for collab cycle.

saderagsdale commented 1 year ago

Moving to icebox

Midge-dev commented 1 year ago

Closing duplicate

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

saderagsdale commented 1 year ago

@Midge-dev @steele-lm This is not a dupe, just FYI. The scope of this ticket is different than the current functionality. Today Veterans can see what they're uploading, but they can't revisit it. The scope of this ticket is to enable Veterans to review what they've uploaded after submission. We put this stakeholder feature request on hold because it was a lower priority at the time.

steele-lm commented 1 year ago

@saderagsdale The AC in #55213 includes the ability to view document contents, so I think that's where the confusion might have originated. It sounds like that AC should be removed from 55213 because it's covered in this ticket -- is that correct?

saderagsdale commented 1 year ago

@steele-lm If your team has picked this back up, then we may want to have a refinement discussion so I understand the context of what you're doing.