-
Notifications
You must be signed in to change notification settings - Fork 336
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 medicine_administration field #1025
Conversation
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Codecov ReportBase: 55.97% // Head: 55.99% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #1025 +/- ##
==========================================
+ Coverage 55.97% 55.99% +0.02%
==========================================
Files 171 171
Lines 8555 8559 +4
Branches 1459 1474 +15
==========================================
+ Hits 4789 4793 +4
Misses 3717 3717
Partials 49 49
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
@@ -435,6 +435,10 @@ class InsulinIntakeFrequencyType(enum.Enum): | |||
default=list, validators=[JSONFieldSchemaValidator(NURSING_PROCEDURE)] | |||
) | |||
|
|||
medicine_administration = JSONField( |
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.
can we have a JSON schema for this as well? having a schemaless field might get hard for standardization at some point.
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.
The JSON schema for this is going to be very complex. It has still not been finalized. Once it will, I will add the schema in the backend as another PR.
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.
okays, make an issue and tag it here even if this PR is merged
A JSON field to host data for Medicine Administration Record in Daily rounds, a new proposed feature as mentioned in ohcnetwork/care_fe#3437