Skip to content
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.

Sign up for GitHub

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

Jump to bottom

(bug) Patient Checkin Report is broken #770

Closed
jniles opened this issue Oct 4, 2016 · 2 comments
Closed

(bug) Patient Checkin Report is broken #770

jniles opened this issue Oct 4, 2016 · 2 comments
Assignees
@jniles
Labels
Bug

Comments

@jniles
Copy link
Collaborator

jniles commented Oct 4, 2016

The Patient Checkin report no longer works via the button on the Patient Record page. It is likely simply an outdated link or outdated report module.

Steps to Reproduce

  1. Navigate to any patient record
  2. Click the Print link on the Patient Checking box

Expected Result

  1. A print dialog should appear with the PDF print link

Actual Result

  1. A 404 error on http://localhost:8080/reports/patients/:uuid/checkins?renderer=pdf.
@jniles jniles added the Bug label Oct 4, 2016
@jniles jniles added this to the Medical/Hospital Impementation milestone Oct 4, 2016
@jniles
Copy link
Collaborator Author

jniles commented Oct 6, 2016

Upon further exploration, it looks like this report was never updated to the new report manager syntax introduced in #741.

@jniles jniles self-assigned this Oct 10, 2016
@jniles
Copy link
Collaborator Author

jniles commented Oct 10, 2016

I'll take this one.

jniles pushed a commit to jniles/bhima that referenced this issue Oct 10, 2016
fix(checkins): fix patient checkin report
139491f 
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.
@jniles jniles mentioned this issue Oct 10, 2016
Merged
4 tasks
jniles pushed a commit to jniles/bhima that referenced this issue Oct 10, 2016
fix(checkins): fix patient checkin report
ae2b500 
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.
jniles pushed a commit to jniles/bhima that referenced this issue Oct 11, 2016
fix(checkins): fix patient checkin report
cfda425 
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.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jniles jniles

Labels
Bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jniles