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

Household pregnancy by age group | Measure Report from Composition #1404

Merged
merged 24 commits into from
Jul 4, 2022

Conversation

maimoonak
Copy link
Contributor

@maimoonak maimoonak commented Jul 1, 2022

Fixes #1391

Checklist

  • I have written Unit tests for any new feature(s) and edge cases for bug fixes
  • I have added any strings visible on UI components to the strings.xml file
  • I have updated the CHANGELOG.md file for any notable changes to the codebase
  • I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
  • I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

@codecov
Copy link

codecov bot commented Jul 1, 2022

Codecov Report

Merging #1404 (714a835) into main (12f1af2) will decrease coverage by 0.09%.
The diff coverage is 61.53%.

@@             Coverage Diff              @@
##               main    #1404      +/-   ##
============================================
- Coverage     62.15%   62.06%   -0.10%     
- Complexity      739      742       +3     
============================================
  Files           136      137       +1     
  Lines          4838     4858      +20     
  Branches        765      771       +6     
============================================
+ Hits           3007     3015       +8     
- Misses         1399     1411      +12     
  Partials        432      432              
Flag Coverage Δ
engine 62.06% <61.53%> (-0.10%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...e/configuration/view/MeasureReportConfiguration.kt 36.84% <58.33%> (ø)
...ngine/configuration/app/AppConfigClassification.kt 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 12f1af2...714a835. Read the comment docs.

Copy link
Member

@pld pld left a comment

Choose a reason for hiding this comment

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

what a wonderful pr

@maimoonak maimoonak merged commit 382dc68 into main Jul 4, 2022
@maimoonak maimoonak deleted the 1391_hh_preg_measure branch July 4, 2022 06:29
qiarie pushed a commit that referenced this pull request Jan 15, 2025
…1404)

* Sick child visit

* Referral flow | Unit tests

* Unit test fixes | Structure map fixes

* ANC visit plan

* Unit test fixes

* ANC careplan completion and testing

* Update sdk versions

* Move hardcoded condition to regex | ReAdd support v4

* Fix failing tests

* Ignore test passing local failing CI

* Fix failing test

* Household members by age and gender

* Use group members for count instead of all poatients

* Spotless fix

* Load measure and lib data from database | allow assets as well

* Household Pregancy Indicator

* Fix test covergae
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.

[Quest/eCBIS] - Configure Quest app to view measure report for Indicator 2 - Pregnancy Count
2 participants