-
Notifications
You must be signed in to change notification settings - Fork 300
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
Add validation for max dosage in PRN prescription #2383
Comments
I would like to work on this issue. |
Hi @sainak I would like to work on this issue. |
Update I am working on it. |
@sainak Can I add validation to the Prescription model itself? |
@sainak need some help regarding api call? |
@sainak @rithviknishad Please review is this the required output, should I start writing the tests and please guide me to do so. |
Yup, lgtm |
@rithviknishad Please guide me how to write the tests and what exactly is required for the pr to get merged |
@rithviknishad @sainak Please review my PR for the issue. |
Can I pick this up ? There haven't been any response since last 2 weeks |
I was busy with my exams and have already raised the pr just some issues with that will try to resolve. |
Add backend validation to check that max dosage in 24 is greater than or equal to base dosage
https://github.com/coronasafe/care/blob/68a3a8281cf72eb859b5951ffbadda0399c3010f/care/facility/api/serializers/prescription.py#L97-L151
The text was updated successfully, but these errors were encountered: