search

ClinGen / clincoded

This GCI/VCI 1.0 platform has now been retired, and replaced with our new 2.0 platform:
https://github.com/ClinGen/gene-and-variant-curation-tools/issues
MIT License
25 stars 9 forks source link

Text changes for PHI Disclaimer when starting Interpretation in the VCI #2123

Closed cgpreston closed 4 years ago

cgpreston commented 4 years ago

Currently when a VCI user clicks on the "Interpretation" button they are asked to agree to a PHI Disclaimer

The following changes should be made to this pop-up:

  1. Rename to "Submission Policy Agreement"
  2. Replace the current language with the following:

    Users planning to submit evidence to the ClinGen curation interface(s) acknowledge and agree to the following"

    1. Any data entered into the VCI may be made publicly accessible, either through the VCI directly or by subsequent transfer to other public resources (ClinVar, ClinGen Evidence Repository, etc.);
    2. All unpublished patient-specific data entered into the VCI, which is not explicitly consented for public sharing, should be the minimum necessary to inform the clinical significance of genetic variants; and
    3. Data entered into the VCI should not include protected health information (PHI) or equivalent identifiable information as defined by regulations in your country or region;

    Do you agree to these terms?

Keep the Agree/Disagree button

cgpreston commented 4 years ago

This is the current view:

Screen Shot 2020-01-24 at 12 15 56 PM

This is a rough mockup of the intended new view:

Screen Shot 2020-01-24 at 12 17 07 PM
cgpreston commented 4 years ago

@RaoMadhavrao, @wrightmw , this is 1 of 2 tickets we hope to implement soon for the DAPC/VCI text changes that were approved by the Steering Committee today.

shaungc-su commented 4 years ago

@cgpreston I'll will be working on this issue. Thanks for posting the screenshot, it's very helpful to locate the issue.

As I search in the code base, I found another place containing this text - it's under variant central -> Interpretation -> segregation. I wonder if this place needs the same change as well, but I'm not sure where is it used in the website. I'm trying to find out how to get there in the website, probably something related to segregation.

cgpreston commented 4 years ago

@shaungc-su , awesome! With regards to your comment, the same text currently appears when the variant curation record is initiated AND at the top of the segregation tab. It's possible that what you're seeing is related to #2124, which is the text at the top of the Case/Segregation tab in the VCI.

shaungc-su commented 4 years ago

@cgpreston Nice! Thought it was in modal, now I see the top text in segregation tab. I will have the text change applied to both places.

wrightmw commented 4 years ago

Was included in R31 release.