icgc-argo / dac-ui

Development of the ICGC ARGO Data Access Control UI
https://daco.icgc-argo.org/
GNU Affero General Public License v3.0
0 stars 0 forks source link

[Data Hookup+UI] Admin Approval flow + Application state #61

Closed kcullion closed 3 years ago

kcullion commented 3 years ago

Goal

To complete the admin approval flow and transition the application to the correct state.

Note, the approval email that gets triggered will be built in the email epic here: https://github.com/icgc-argo/dac-ui/issues/68 (All emails to be built and triggered in this epic: https://github.com/icgc-argo/dac-ui/issues/65)

Endpoint to use

https://github.com/icgc-argo/dac-api/issues/16

Expected Outcomes

Confirmation Modal

Application state after approved

image

Application state after approved for DACO Admin

Section C

Section E

will be hooked up in this ticket https://github.com/icgc-argo/dac-ui/issues/58

samrichca commented 3 years ago

Some work can't be completed right now because the following aren't implemented

Things I can't do yet:

  1. Section E is locked if the applicant had chosen option 1 (do not require an ethics letter) and unlocked if the applicant had chosen option 2, that they require a letter.
  2. If the applicant had chosen option 2 (required an ethics letter), show the ethics uploads that were originally submitted but hide the delete (trashcan) icons for those, as seen: https://zpl.io/beP4zMY
  3. Disable option 1 (they cannot go from requires a letter to doesn't require a letter), as seen: https://zpl.io/beP4zMY
  4. If the applicant uploads a new ethics pdf, that pdf is shown in the table and is removable, as seen: https://zpl.io/blOm11X
kcullion commented 3 years ago

Thanks @samrichca I'll move these items to a new ticket

kcullion commented 3 years ago

update: there was a ticket for it:https://github.com/icgc-argo/dac-ui/issues/58

samrichca commented 3 years ago

@kcullion even better! thanks for finding that

kcullion commented 3 years ago

@samrichca yes I"m glad you didn't waste time on that! forgot that I put that as part of your ticket

rosibaj commented 3 years ago

Tested Jul 12, 2021

lgtm thank you @samrichca!!