-
Notifications
You must be signed in to change notification settings - Fork 224
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
(fix) O3-4039: Enable backdating a prescription's start date #2057
base: main
Are you sure you want to change the base?
Conversation
This PR fixes an issue where the Start date field of the drug order form was not being set correctly. The default value for the field was being set to the current date. Instead, the field should be set to the value of the medication order's `dateActivated` property (when editing an existing order) or the current date (when creating a new order). This PR also refactors the MedicationsDetailsTable component to leverage the new MedicationsDisplaySection sub-component. The idea with this refactor is to clean up the logic for rendering the medications section and to make the component more modular and easier to maintain. Ticket: https://openmrs.atlassian.net/browse/O3-4039
Size Change: -31.4 kB (-0.21%) Total Size: 14.8 MB
ℹ️ View Unchanged
|
Thanks, sir @denniskigen. We got this request from the field and am glad you have provided a fix. I would request that we also fix it for lab orders. Most orders done retrospectively pick the current date and it becomes an issue for sensitive tests i.e. viral load and CD4. |
Any pointers, @ibacher? |
I don't think this issue is fixable with the order basket working the way that it does, i.e., creating an "Orders" encounter and using that. Basically, this ticket requires some kind of RDE functionality to pre-exist. The "minimal" thing to do here would be to move the start time of the orders encounter, but that cannot be moved before the start of the current visit and could potentially raise other issues if there are orders already attached to the encounter. Instead of trying to fix things piecemeal like this, my vote is we make "support for RDE" into a community priority and start on that. |
Requirements
Summary
This PR fixes an issue where the Start date field of the drug order form was not being set correctly. The default value for the field was being set to the current date. Instead, the field should be set to the value of the medication order's
dateActivated
property (when editing an existing order) or the current date (when creating a new order).This PR also refactors the MedicationsDetailsTable component to leverage the new MedicationsDisplaySection sub-component. The idea with this refactor is to clean up the logic for rendering the medications section and to make the component more modular and easier to maintain.
Screenshots
Related Issue
https://openmrs.atlassian.net/browse/O3-4039
Other