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.

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

Revise the Active in PMTCT column in the Active on ART Patients Linelist #2023

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

gabriel090
Copy link

Revise the Active in PMTCT column in the Active on ART Patients Linelist to show PMTCT mothers who are pregnant or breastfeeding.

Replace the current ‘Yes’ with either the mother is pregnant or Breastfeeding. If the mother is not active in PMTCT retain ‘No’.

@ojwanganto
Copy link

Should this query check for the last records of the patient from the associated flat tables? I have a feeling that this can pick a mid-table status and not the last. Please review and provide feedback

@gabriel090
Copy link
Author

Should this query check for the last records of the patient from the associated flat tables? I have a feeling that this can pick a mid-table status and not the last. Please review and provide feedback

I have modified. Thanks

Copy link

@patryllus patryllus left a comment

Choose a reason for hiding this comment

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

If this is a date base report - you should use mid(max) for the given visit date as using only max you will get the same max for any date

…ist to show PMTCT mothers who are pregnant or breastfeeding.

Replace the current ‘Yes’ with either the mother is pregnant or Breastfeeding. If the mother is not active in PMTCT retain ‘No’.
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