ClinGen / gene-and-variant-curation-tools

ClinGen's gene and variant curation interfaces (GCI & VCI). Developed by Stanford ClinGen team.
https://curation.clinicalgenome.org/
MIT License
3 stars 1 forks source link

Capture and disseminate VCEP curation process reasons #318

Closed wrightmw closed 3 months ago

wrightmw commented 1 year ago

At point of publishing, capture reasons for the curation process.

See mockup here: https://docs.google.com/presentation/d/1TvGdv4u1YM7piqA7JzkAGXPcF46_3W7ma8qduGUOlXk/edit?usp=sharing

Outstanding issues to be discussed with invested users/proxies:

cgpreston commented 1 year ago

Follow up slides for further proxy feedback here: https://docs.google.com/presentation/d/1zFUQpywVPHJ4lEwGrwKzLMbeEjLFqlBRCaYJ7TrQ8tg/edit#slide=id.g2542a3fea59_0_0

cgpreston commented 11 months ago

Follow up from Tools call on 7/20:

  1. Streamline the top menu of both GCI/VCI selections to "Curation" (major update, first digit), "Recuration" (Major update, first digit), and "Administrative update" (minor update, second digit).
  2. "Gene curation express" to become a sub option of Recuration
  3. "Republish due to technical issues" to become a subselection of "Administrative update", with a 3rd digit incrementation).

Both tool tips and a link to the SOP/selection guidance is desired to help users. We need to think about what's going to happen when a user selects the wrong option.

"New Curation" is not allowed in combination with any other selections, the other selections are allowable in combinations.

cgpreston commented 9 months ago

@markmandell, two (hopefully small) changes have been decided based on proxy feedback:

For both the VCI and GCI publishing modals please:

  1. Remove the field/text box currently labeled "Additional comments about classification (optional):", this is no longer needed.
  2. Rename the field/text box current labeled "Additional comments about publishing reasons (optional):"
    • new name: "Comments about publishing reasons (optional, for internal use only):"
wrightmw commented 9 months ago

@cgpreston

Do we want to just rename the currently labeled "Additional comments about classification (optional)" to "Comments about publishing reasons (optional, for internal use only)"?

And then just not implement the new text box?

cgpreston commented 9 months ago

@wrightmw , I think it's up to @markmandell, which is easiest to do, but yes, we just need one box. And I like your language suggestion for the box label, thanks!