-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
fix(medication): set correct quantity and value for new medications #2554
fix(medication): set correct quantity and value for new medications #2554
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/hospitalrun/hospitalrun-frontend/7c4YFBXAQTwUWfHThAiwU6yZYBwK |
7af199d
to
6266148
Compare
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.
Code changes LGTM as they appear to resolve the issue. May you fix the lint errors in src/medications/requests/NewMedicationRequest.tsx andwe should be all set.
@blestab All set! I had to fix the extends for the eslint rules for |
These rules conflict and eslint-config-prettier is set to disable conflicting prettier rules if extends includes the plugins. https://github.com/prettier/eslint-config-prettier#installation > If you extend a config which uses a plugin, it is recommended to > add "prettier/that-plugin" (if available).
6266148
to
6371239
Compare
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
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 would have expected a test to be modified/added to show that this is working
Fixes #2492.
Fixes setting the quantity and value properties on the medication model via the form
NewMedicationRequest
.Changes proposed in this pull request: