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

populate mamba reports #1847

Merged
merged 9 commits into from
May 24, 2024
Merged

populate mamba reports #1847

merged 9 commits into from
May 24, 2024

Conversation

lucyjemutai
Copy link
Contributor

@lucyjemutai lucyjemutai commented May 16, 2024

Requirements

  • This PR has a title that briefly describes the work done including the ticket number. If there is a ticket, make sure your PR title includes the ticket number in the format OHRI-123 My PR title.
  • My work includes tests or is validated by existing tests.

This pr address the following;
Populate 3 mamba report for the MNCH Summary ;

  1. no_of_anc_visits
  2. mother_status
  3. estimated_date_of_delivery
  4. mother_hiv_status

Screenshots

image

Related Issue

Other

Copy link
Contributor

@CynthiaKamau CynthiaKamau left a comment

Choose a reason for hiding this comment

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

Thanks @lucyjemutai , i left a comment


useEffect(() => {
if (patient) {
const lastIdentifier = patient.identifier
Copy link
Contributor

Choose a reason for hiding this comment

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

I have some reservations on this, the last pTracker id will probably have the latest date and one that is no longer valid has an end date., unless a patient can have more than one pTrackerId.

You can have a look at this file, they are fetching identifiers based on types : https://github.com/openmrs/openmrs-react-components/blob/0a739855da0262b6cb79f7e27ae5ff62932232f4/src/domain/patient/patientUtil.js#L108

patient.identifiers.find((identifier) => identifier.type.id === identifiersTypes.PTrackerIdentifierType);
should work

@lucyjemutai lucyjemutai requested a review from CynthiaKamau May 21, 2024 12:07
Copy link
Contributor

@arodidev arodidev left a comment

Choose a reason for hiding this comment

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

Thanks @lucyjemutai, left some review comments.

@lucyjemutai lucyjemutai requested a review from arodidev May 21, 2024 20:51
@lucyjemutai
Copy link
Contributor Author

@arodidev please re-review again

@lucyjemutai lucyjemutai requested a review from arodidev May 24, 2024 12:30
Copy link
Contributor

@arodidev arodidev left a comment

Choose a reason for hiding this comment

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

Thanks @lucyjemutai, LGTM

@arodidev arodidev merged commit 26b38d0 into UCSF-IGHS:dev May 24, 2024
5 checks passed
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