Skip to content
This repository was archived by the owner on Jul 31, 2023. It is now read-only.

Conversation

@smalho01
Copy link

@smalho01 smalho01 commented Mar 3, 2022

No description provided.

priorAuthBundle.signature = signature;
priorAuthBundle.entry.unshift({ resource: priorAuthClaim });

const specialtyRxBundle = JSON.parse(JSON.stringify(priorAuthBundle));

Choose a reason for hiding this comment

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

Is there a reason to keep the priorAuthBundle around? Can we just remove it and use the specialtyRxBundle exclusively or does REMS have some use for the priorAuthBundle?

Copy link
Author

Choose a reason for hiding this comment

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

I think its nice to show the synergy between the two but its ultimately not a part of this use case

const specialtyRxBundle = JSON.parse(JSON.stringify(priorAuthBundle));
specialtyRxBundle.type = "message";
if (this.makeReference(priorAuthBundle, "MedicationRequest")) {
const pharmacy = {

Choose a reason for hiding this comment

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

This is probably a broader question but how is DTR supposed to know where to send this bundle? Will it be a config option or is it supposed to be dynamic and passed in when the app launches? Theoretically this resource or the MessageHeader should have the info on where to send it.

Copy link
Author

Choose a reason for hiding this comment

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

for now hard code it to send to the the PIS system at the following url:
localhost:3010/api/doctorOder/FHIR

}

const specialtyRxMessageHeader = {
resourceType: "MessageHeader",

Choose a reason for hiding this comment

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

MessageHeader is missing required source field

Copy link
Author

Choose a reason for hiding this comment

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

discussed at meeting, will be added in as part of REMS-86

@smalho01 smalho01 merged commit e22353e into master Mar 8, 2022
@smalho01 smalho01 deleted the REMS-85 branch November 2, 2022 17:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants