Skip to content

DRC-7: Add missed appointments report as initial work for Treatment interruptions and loss of follow-up#8

Open
enyachoke wants to merge 1 commit intomainfrom
DRC-7-report
Open

DRC-7: Add missed appointments report as initial work for Treatment interruptions and loss of follow-up#8
enyachoke wants to merge 1 commit intomainfrom
DRC-7-report

Conversation

@enyachoke
Copy link
Copy Markdown
Contributor

No description provided.

@enyachoke enyachoke requested a review from reagan-meant May 19, 2025 11:28

@Override
public String getName() {
return "Missed Appointments Report";
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't these return the translatable strings (as in MessageUtil.translate(...))?

JOIN obs o ON p.patient_id = o.person_id
JOIN person per ON p.patient_id = per.person_id
JOIN person_name pn ON per.person_id = pn.person_id
WHERE o.concept_id = 211
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because of the way metadata is loaded, it is not safe to rely on concept_ids being consistent across instances of the same system.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Things I'm not quite clear how this report handles:

  1. Isn't there potentially some duplication between the "Next Appointment Date" and actual schedule appointments? How do we deduplicate those?
  2. Isn't this just a list of appointments? I.e., I don't see how this filters out appointments that have been attended (I wouldn't expect that this is done via voiding an appointment).
  3. How do we filter patients who have either died or have transferred out of care?


@Override
public String getDescription() {
return "Missed Appointments Report";
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We do the translation here too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants