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

Change names of Affiliations to match new agreed format #1697

Closed wrightmw closed 6 years ago

wrightmw commented 6 years ago

All Affiliation names in the interfaces will need to be changed before the next release to match a new agreed name format. This will mirror the website display, short version. Name format: [Group Name] EP
Examples: Hearing Loss EP, Autism and Intellectual Disability EP​

Dependency: The full list of new names will be provided by the Geisinger team.

wrightmw commented 6 years ago

@ErinRiggs We need to have the agreed new names for the current Affiliations by the end of business on Monday so that they can be included in the next release. It's fairly important that these are sorted when the publication feature goes live in the next release because summaries on the website will include the names from the GCI. Should we have to change the names of these Affiliations at a later date then some summaries will have the old names and some the new, unless they are then re-published to the website. So, it would definitely be preferable to get the new names into this release.

Please see below a table of the old/current names in the GCI/VCI and new names I have transposed based on your formula that was agreed in the GCI WG call. Would it be ok to go ahead with these new names if we don't hear otherwise? Please advise any changes you think we should make. Thanks.

screen shot 2018-06-18 at 9 13 55 am

jimmyzhen commented 6 years ago

Addressing this ticket in the #1695 branch.

jimmyzhen commented 6 years ago

Hi @selinad, @wrightmw,

Please review the following instance for the fixes/changes requested in this ticket: https://1695-jz-exac-fix.demo.clinicalgenome.org

selinad commented 6 years ago

@jimmyzhen looking good overall, with what I think are a few needed updates:

@wrightmw please let me know if above edits are correct. thx!

wrightmw commented 6 years ago

@jimmyzhen Yes, I agree with those edits suggested by @selinad

wrightmw commented 6 years ago

@jimmyzhen In addition to the above edits, there is one further edit, which I have wrong in the table above. "Intellectual Disability & Autism EP" should be:

jimmyzhen commented 6 years ago

@selinad, @wrightmw,

Great catch on my mistakes! Thank you both!

I have corrected them and updated the instance. Please give it another look when you get a chance.

selinad commented 6 years ago

@jimmyzhen all looks good now! @wrightmw I've pasted in the corrected table for reference (please check). Thank you! image

wrightmw commented 6 years ago

@selinad All looks good to me!

selinad commented 6 years ago

This all looks correct, @jimmyzhen ! Also includes Mitochondrial Diseases EP now:

image

image

wrightmw commented 6 years ago

Thanks to @selinad manually typing in this list of AffiliationIDs into her name, I was able to copy across these into my name super-quick. All new names are present and correct @jimmyzhen