-
Notifications
You must be signed in to change notification settings - Fork 211
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
(refactor) O3-2849: Refactor appointments mutator functions #993
Conversation
@@ -3,7 +3,7 @@ import { type AppointmentPayload } from '../../types'; | |||
import dayjs from 'dayjs'; | |||
import { omrsDateFormat } from '../../constants'; | |||
|
|||
// TODO we will refactor all this stuff to use the cancel functionality from patient-chart (or vice versa, but will move into the forms directory regardless) | |||
// TODO we should refactor all this stuff to use the cancel functionality from patient-chart (or vice versa, but should move into the forms directory regardless) |
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.
Tangential tweak to a comment... :)
Size Change: -49.3 kB (-2%) Total Size: 2.96 MB
ℹ️ View Unchanged
|
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.
Nice refactoring.
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.
Nice work, @mogoodrich.
); | ||
|
||
return { | ||
mutate: mutateAppointments, |
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.
Nit: Might as well call this mutateAppointments
to avoid aliasing the function name downstream.
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.
Fair enough
Thanks all! Merging this in. |
Requirements
Summary
This refactors several of the mutator functions that get run when creating appointment records.
Screenshots
Related Issue
Other