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 202 forks source link

[CST] Figure out what post-upload state should look like in the Files tab #75136

Open skylerschain opened 8 months ago

skylerschain commented 8 months ago

Value Statement

As a Veteran I want to be able to upload evidence and have it be clear what I'm uploading So that I can be assured the right files are getting submitted to the claim


Engineering Notes

This ticket relates to https://github.com/department-of-veterans-affairs/vets-website/pull/27749.

(2/2/24) @pmclaren19 spoke with the platform design team and there is a multiple prop in the works for the V3 VAFileInput but it is still being worked on. We expect it to be release in 2 - 4 weeks and then we can update this ticket design.

New Feature

We're reformatting our Files tab and updating the File Uploader component to the new V3 version.

Screenshot 2024-02-01 at 12.59.07 PM.png

Design

As part of this, we need to mock up what the post-upload state should look like. In the version @pmclaren19 created initially, there are two different confirmation elements showing the user which file they uploaded. This feels unnecessary. Design will work on figuring out a more elegant solution.

Current implementation

Screenshot 2024-02-01 at 9.36.04 AM.png

Open questions

Current Assumptions

Tasks

Definition of Done

Acceptance Criteria

skylerschain commented 7 months ago

@pmclaren19's Slack thread for reference: https://dsva.slack.com/archives/C01DBGX4P45/p1709219006502679?thread_ts=1709154979.517099&cid=C01DBGX4P45

jacobworrell commented 6 months ago

@skylerschain @pmclaren19 do we still need this ticket or is the assumption that the v3 component will eventually include this functionality?

pmclaren19 commented 6 months ago

This still needs to be worked on once platform has made their changes as we will still need to make design changes and eng changes to determine how we want the file upload to look.

[PLATFORM DESIGN BUG] - Unable to upload multiple files at the same time https://github.com/department-of-veterans-affairs/vets-design-system-documentation/issues/2549 has been split into 3 tickets...

[PLATFORM DESIGN BUG] - Unable to delete a file once its been added https://github.com/department-of-veterans-affairs/vets-design-system-documentation/issues/2511 this work has been added into this ticket...

[PLATFORM DESIGN BUG] - There isnt a card design that matches and works well with this component https://github.com/department-of-veterans-affairs/vets-design-system-documentation/issues/1578 this work has been added to this ticket...

jacobworrell commented 2 months ago

@pmclaren19 @skylerschain is this something we need to work on now?

pmclaren19 commented 2 months ago

@jacobworrell @skylerschain now that va-file-input-multiple exists I dont think we need this ticket anymore and it is replaced with https://github.com/department-of-veterans-affairs/va.gov-team/issues/87835. LMK your thoughts!