-
Notifications
You must be signed in to change notification settings - Fork 248
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
O3-2891: Refactor Patient Appointments ESM into Appointments ESM #1739
Conversation
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.
LGTM. (That was easy...)
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.
Thanks, @mogoodrich!
Thanks @chibongho @ojwanganto and @denniskigen .... the checks appear to be hung so I may close and reopen to try to resolve and then push the new PR through without review... :) |
Size Change: -932 kB (-8%) ✅ Total Size: 10.8 MB
ℹ️ View Unchanged
|
Fixed a couple things I missed that were causing errors, merging this in now, thanks all! |
This basically removed the entire patient-appointments ESM, needs to merged in with it's partner PR that adds this functionality to the appointments app in the patient management monorepo, see:
https://openmrs.atlassian.net/browse/O3-2891
Requirements
Summary
Screenshots
Related Issue
Other