OpenMRS: Add fully-featured fhir namespace
@josephjclark is already working on this.
Since Jan 15, 2025.
- Dominant language
- JavaScript
- Stars
- 24
- Forks
- 41
- Avg merge
- 3d 13h
- Merged PRs (30d)
- 12
Description
See parent issue #887 for details.
- Import the fhir v4 adaptor
- Re-export basically all of it through the fhir` namespace. So we should be able to do this:
fhir.create(builders.patient({ identifier: $.patient.id })
- Remove the fhir.utils file (any fhir utils should just go in fhir.js)
The adaptor should use state.configuration.instanceUrl as the base url for all fhir requests, and should handle auth consistently. I am not totally sure how that will work against the wrapped fhir functions yet.
Include a fhir.request function to make an arbitrary request against the fhir root of the isntanceurl. Do not add get/put/etc helpers.
I think the fhir builders should use the root namespace, so builders.patient, and not the fhir namespace, fhir.builders.patient. Just because it's really annoying
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.