-
Notifications
You must be signed in to change notification settings - Fork 102
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(checkins): fix patient checkin report #795
fix(checkins): fix patient checkin report #795
Conversation
139491f
to
ae2b500
Compare
@mbayopanda, can I get a review? |
This commit updates the patient checkin report to the latest report standard. It adds integration tests for the patient checkin report route. Closes IMA-WorldHealth#770.
ae2b500
to
cfda425
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.
👍 All things in this PR are good, it just notice that the pdf report of checkin it is not translated.
Other thing is, the french transalation of checkin
(verification) it is not well for the context, i think we have to choose a good name for example Visite
and Signaler visite du patient
for text on the checkin button.
* @requires renderers/pdf | ||
* @requires lib/db | ||
* @requires lib/ReportManager | ||
* @requires Patients |
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.
👍
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've created an issue to track the updates suggested by @mbayopanda #811. I suggest that this pull request is merged with translations to follow. |
This commit updates the patient checkin report to the latest report standard. It adds integration tests for the patient checkin report route.
Closes #770.
Thank you for contributing!
Before submitting this pull request, please verify that you have:
For a more detailed checklist, see the official review checklist that this PR will be evaluated against.
Ensuring that the above checkboxes are completed will help speed the review process and help build a stronger application. Thanks!