Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Phase 5: Add fully-featured fhir namespace #893

Open
aleksa-krolls opened this issue Jan 9, 2025 · 1 comment
Open

Phase 5: Add fully-featured fhir namespace #893

aleksa-krolls opened this issue Jan 9, 2025 · 1 comment
Assignees

Comments

@aleksa-krolls
Copy link
Member

aleksa-krolls commented Jan 9, 2025

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

@martalovescoffee
Copy link

@hunterachieng could you please add an estimate to this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: New Issues
Development

No branches or pull requests

4 participants