Merge pull request #417 from nf-osi/allaway-patch-2

make diagnosis required attribute
nf-osi · Apr 8, 2024 · cffba6b · cffba6b
2 parents b166766 + 1dfdbaa
commit cffba6b
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/NF.jsonld b/NF.jsonld
@@ -1904,7 +1904,7 @@
     "schema:isPartOf" : {
       "@id" : "http://schema.biothings.io/"
     },
-    "sms:required" : "sms:false",
+    "sms:required" : "sms:true",
     "schema:rangeIncludes" : [ {
       "@id" : "bts:Neurofibromatosistype1"
     }, {
@@ -1923,6 +1923,8 @@
       "@id" : "bts:Schwannomatosis-NEC"
     }, {
       "@id" : "bts:SporadicSchwannoma"
+    }, {
+      "@id" : "bts:NotApplicable"
     } ],
     "rdfs:label" : "diagnosis",
     "rdfs:comment" : "Diagnosis for the individual given signs and symptoms. Use the most specific diagnosis term that applies.",

diff --git a/modules/Sample/Diagnosis.yaml b/modules/Sample/Diagnosis.yaml
@@ -26,6 +26,7 @@ enums:
           Most schwannomas originate from the eighth cranial nerve, whereas neurofibromas more commonly arise along the spinal nerve roots.
         notes:
         - Not in NCIT as of 2023-06-07
+      Not Applicable:
 
   AgeGroupEnum:
     permissible_values:

diff --git a/modules/props.yaml b/modules/props.yaml
@@ -342,7 +342,7 @@ slots:
   diagnosis:
     description: Diagnosis for the individual given signs and symptoms. Use the most specific diagnosis term that applies.
     range: DiagnosisEnum
-    required: false
+    required: true
   diagnosisAgeGroup:
     description: Age group of the individual at the time of diagnosis.
     range: AgeGroupEnum