-
Notifications
You must be signed in to change notification settings - Fork 14
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
update setup bot #8
Conversation
}, | ||
], | ||
}, | ||
'Practitioner/practitioner' |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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' |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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` |
There was a problem hiding this comment.
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' |
There was a problem hiding this comment.
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' |
There was a problem hiding this comment.
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' |
There was a problem hiding this comment.
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' |
There was a problem hiding this comment.
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' |
There was a problem hiding this comment.
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' |
There was a problem hiding this comment.
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' |
There was a problem hiding this comment.
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' |
There was a problem hiding this comment.
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' |
There was a problem hiding this comment.
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' |
There was a problem hiding this comment.
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' |
There was a problem hiding this comment.
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' |
There was a problem hiding this comment.
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' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same issue
weight, | ||
message | ||
}; | ||
} |
There was a problem hiding this comment.
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'; |
There was a problem hiding this comment.
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
No description provided.