-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Added Endpoints for Prescription/MedicationRequest Resource #3821
Conversation
Signed-off-by: Yash Bothra <yashrajbothra786@gmail.com>
Signed-off-by: Yash Bothra <yashrajbothra786@gmail.com>
Signed-off-by: Yash Bothra <yashrajbothra786@gmail.com>
Signed-off-by: Yash Bothra <yashrajbothra786@gmail.com>
Signed-off-by: Yash Bothra <yashrajbothra786@gmail.com>
The callback in find_code_dynamic doesn't support target. Should we add sel_related_target like this
Or is there some other way to do this? If needed to get something to work, then go for it :) |
Signed-off-by: Yash Bothra <yashrajbothra786@gmail.com>
travis is pointing out a php deprecated error for 7.4 and 8.0:
|
LEFT JOIN users AS practitioner | ||
ON practitioner.id = prescriptions.provider_id | ||
LEFT JOIN drugs AS drug | ||
ON drug.drug_id = prescriptions.drug_id"; |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
hi @yashrajbothra , Just let me know when this is ready for final testing before bringing into codebase. thanks! -brady |
Hii @bradymiller , All Endpoints are testing well. But the frontend part is not testing well because of the callback |
Signed-off-by: Yash Bothra <yashrajbothra786@gmail.com>
Signed-off-by: Yash Bothra <yashrajbothra786@gmail.com>
Hello @bradymiller Sir, Pushed a fix for Target Element and Added Limit in the Popup. Kindly Review :) Thanks :) |
hi @yashrajbothra ui and rxnorm stuff is testing well. I'm assuming the api endpoints are testing well for you, so bringing this in. thanks for the FHIR endpoints! -brady |
Fixes #
Short description of what this resolves:
Changes proposed in this pull request: