OpenFn / OpenFn/adaptors

OpenMRS: Add fully-featured fhir namespace

Open
#893 1 comment 0 reactions 1 assignee View on GitHub

@josephjclark is already working on this.

Since Jan 15, 2025.

DevX
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

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.