search

BRCAChallenge / BRCAMobile

0 stars 0 forks source link

variants with 'Uncertain' pathogenicity crash the app #16

Closed falquaddoomi closed 6 years ago

falquaddoomi commented 6 years ago

BRCA2 c.8755-1G>A returns a pathogenicity of "Uncertain". The entry shows up with no icon in the search results, and accessing the element throws an error in DetailDisplay.js : renderSignificance() (or just crashes on the production build).

falquaddoomi commented 6 years ago

This appears to have two causes: 1) "Uncertain" not being known to the app as a possible status, causing it to default to "Not Yet Reviewed", and 2) the late change to having all the pathogenicities be lowercase client-side, which caused "Not Yet Reviewed" to not match anything, producing a null value. This causes the results list to simply not display any icon, but the details view crashes on attempting to reference a subfield in the null value.