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

Add resources to DTR Bundle #188

Merged
merged 4 commits into from
Nov 25, 2024
Merged

Add resources to DTR Bundle #188

merged 4 commits into from
Nov 25, 2024

Conversation

elsaperelli
Copy link
Contributor

Summary

Adds resources to the existing dtr_bundle_patient_pat015.json file in order to test the DTR Light EHR Suite (this PR). The ids for these resources are used in the modified inferno_dtr_server_suite.json present for the DTR Light EHR Suite.

New behavior

  • Should not change behavior in any other test suites where this is used.

Code changes

  • Adds the following resources to the file:
    • DTR QuestionnaireResponse
    • CRD CommunicationRequest
    • CRD MedicationRequest
    • CRD NutritionOrder
    • CDex Task
    • CRD VisionPrescription
  • Note: these resources were taken from their respective IG examples and modified to fit the existing pat015 patient if necessary.

Testing guidance

  • @tstrass since you originally created this bundle, please make sure these additions don't break anything you use this for 😄
  • Run this branch locally and run the this branch of the davinci-dtr-test-kit, and run the DTR Light EHR Profiles Tests in the DTR Light EHR Test Suite using the Inferno Reference Server preset.

Copy link
Contributor

@karlnaden karlnaden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With local validator issues, I tried using https://validator.fhir.org/. This turned up one issue on the CDex Task. It is timing out on the QuestionnaireResponse, but knowing what the errors were before, I'm more confident that is right.

resources/dtr_bundle_patient_pat015.json Outdated Show resolved Hide resolved
resources/dtr_bundle_patient_pat015.json Outdated Show resolved Hide resolved
resources/dtr_bundle_patient_pat015.json Outdated Show resolved Hide resolved
Copy link
Contributor

@tstrass tstrass left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested with SMART App Suite, and confirmed this does not break anything.

Copy link
Contributor

@karlnaden karlnaden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ran these against the light DTR tests and they look good now.

@elsaperelli elsaperelli merged commit 4887956 into main Nov 25, 2024
3 checks passed
@elsaperelli elsaperelli deleted the fi-3336-dtr-patient-adds branch November 25, 2024 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants