Open kcullion opened 2 years ago
We may need a backend ticket for returning these values just to be consumed by the front end? Also these can the -- and N/A values be generalized in one ticket - they will be displayed in multiple contexts like on the donor entity page, and the file entity page. @blabadi @demariadaniel please advise.
@kcullion @joneubank adding Jon here since he is working on the clinical API and indexing So Jon, the thing here is we need the UI to be able to distinguish between N/A or empty values, and we need this consistent across the platform so my suggestion is to make sure the clinical API enforce these rules above in the ticket and include the right response so the index or anything that queries the API get N/A where applicable.
not sure if you are still in the middle of this or need a new ticket or have something else in mind to avoid encoding business rules in the FE
This may need to be revisited based to make these rules consistent with what's decided for clinical exceptions (eg. using "Unknown" vs. "Not Available").
General Approach for blank fields and hidden fields
(from slack convo March 16)
Not Applicable fields
Show
N/A
for the following cases:[ ] 2. If
vital_status = Alive
, thencause_of_death = N/A
andsurvival_time = N/A
[ ] If
clinical_tumour_staging_system
(primary_diagnosis.tsv) is NOT any edition of the AJCC cancer staging system, thenClinical TNM Category = N/A
For example:[ ] if
pathological_tumour_staging_system
(specimen.tsv) is NOT any edition of the AJCC cancer staging system, thenPathological TNM Category = N/A
[ ] if
recurrence_tumour_staging_system
orposttherapy_tumour_staging_system
(followup.tsv) is NOT any edition of the AJCC cancer staging system, thenRecurrence TNM Category = N/A
andPosttherapy TNM Category = N/A
Not Available fields
--
when a value is missing and doesn't pertain to the above use casesNote for reviewing hidden fields:
Normal
specimen https://github.com/icgc-argo/platform-ui/issues/2127Just for reference
Melanie sent this resource for the meaning behind missing values. Putting here for adding onto this spec https://ena-docs.readthedocs.io/en/latest/submit/samples/missing-values.html