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

update setup bot #8

Merged

Conversation

IvanSakhman
Copy link
Contributor

No description provided.

},
],
},
'Practitioner/practitioner'
Copy link
Member

Choose a reason for hiding this comment

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

I think this ID should be different, something like Practitioner/pid-12345

Copy link
Member

Choose a reason for hiding this comment

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

Is the goal to have one global practitioner? Or one practitioner per patient?

},
],
},
'Schedule/schedule'
Copy link
Member

Choose a reason for hiding this comment

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

Same with this ID, something like Schedule/sched-12345

Copy link
Member

Choose a reason for hiding this comment

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

The syntax here needs to be a search string, not by id. You could search by identifier, such as Schedule?identifier=xyz

},
],
},
`Observation/hemoglobinA1c`
Copy link
Member

Choose a reason for hiding this comment

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

This also needs to be related to patient. I'm not sure the best way to handle this, but will ask for guidance from @codyebberson and @rahul1 - namely this should be something like `Observation\patient.id+hemoglobinA1c

],
subject,
},
'CarePlan/active-care-plan'
Copy link
Member

Choose a reason for hiding this comment

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

Same issue. This also needs to be related to patient. I'm not sure the best way to handle this, but will ask for guidance from @codyebberson and @rahul1 - namely this should be something like `CarePlan\patient.id+antenatal. Also, @rahul1 - we need to upgrade this to something more in line with our recommendations.

],
subject,
},
'CarePlan/completed-care-plan'
Copy link
Member

Choose a reason for hiding this comment

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

Same issue.

}
]
},
'DiagnosticReport/diagnostic-report'
Copy link
Member

Choose a reason for hiding this comment

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

Same issue again

text: '72 HR Fentanyl 0.025 MG/HR Transdermal System',
},
},
'MedicationRequest/123'
Copy link
Member

Choose a reason for hiding this comment

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

Same issue.

text: 'Acetaminophen 325 MG / Oxycodone Hydrochloride 10 MG Oral Tablet [Percocet]',
},
},
'MedicationRequest/stopped-medication'
Copy link
Member

Choose a reason for hiding this comment

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

same issue

text: 'SARS-COV-2 (COVID-19) vaccine, mRNA, spike protein, LNP, preservative free, 100 mcg/0.5mL dose',
},
},
'Immunization/completed-immunization'
Copy link
Member

Choose a reason for hiding this comment

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

Same issue

},
],
});
'Immunization/not-done-immunization'
Copy link
Member

Choose a reason for hiding this comment

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

Same issue.

effectiveDateTime: date.toISOString(),
status: 'final',
},
'Observation/body-temperature'
Copy link
Member

Choose a reason for hiding this comment

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

same issue.

effectiveDateTime: date.toISOString(),
status: 'final',
},
'Observation/height'
Copy link
Member

Choose a reason for hiding this comment

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

same issue.

effectiveDateTime: date.toISOString(),
status: 'final',
},
'Observation/respiratory-rate'
Copy link
Member

Choose a reason for hiding this comment

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

Same issue.

effectiveDateTime: date.toISOString(),
status: 'final',
},
'Observation/heart-rate'
Copy link
Member

Choose a reason for hiding this comment

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

same issue.

effectiveDateTime: date.toISOString(),
status: 'final',
},
'Observation/weight'
Copy link
Member

Choose a reason for hiding this comment

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

same issue.

},
],
},
'Communication/welcome-message'
Copy link
Member

Choose a reason for hiding this comment

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

Same issue

weight,
message
};
}
Copy link
Member

Choose a reason for hiding this comment

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

@codyebberson and @rahul1 - this part is interesting. Not sure the right path here, but this seems sub-optimal

@@ -1,102 +1,546 @@
import { BotEvent, MedplumClient, createReference, getDisplayString } from '@medplum/core';
Copy link
Member

Choose a reason for hiding this comment

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

This file will need to be changed when script is updaed

@codyebberson codyebberson merged commit 7ee5176 into medplum:reshma-setup-bot Jul 21, 2022
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.

None yet

3 participants