omoponfhir / omoponfhir-main

Main OMOPonFHIR Settings
Apache License 2.0
45 stars 15 forks source link

MedicationStatement FHIR required fields #4

Closed eric-chou closed 4 years ago

eric-chou commented 4 years ago

Hello, Thank you again for your work on this project! It has worked very smoothly in providing FHIR resources for our data, but one thing that we were wondering about is that it appears that under the FHIR STU3 specification, the "status" element is not provided when OMOP on FHIR renders the resources for specific dataset. This may become an issue for us since some of the FHIR applications we hope to work with may check for airtight conformance to the STU3 standard, and a missing "required" field such as MedicationStatementStatus could be an obstacle. Thus, we were wondering if there was anything that might be missing in our data that is resulting in this field not being available, or if something else needs to be introduced in the data or in the codebase.

An example of this should be able to be seen at this URL (a screenshot of this resource is at the bottom in case this doesn't load). All of our MedicationStatement resources are available here.

image

myungchoi commented 4 years ago

The status is fixed on Nov 2019: https://github.com/omoponfhir/omoponfhir-omopv5-stu3-mapping/commit/b85802ffcbf18c98a01be4818b289d097c6c31bc

Please see constructFHIR method in OmopMedicationStatement.java

If you update yours, the status will show up. If OMOP drug exposure has stop reason populated, then the status will be changed to stopped and reason is added to note. Otherwise, status is set to active.