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

[Quest/Malawi Core] Android App : FHIR Practitioner details API update #3087

Conversation

calmwalija
Copy link

Description
The application is failing to sync due to practitioner details API update.

Fixes https://github.com/ona-health/mwcore-fhir-resources/issues/248

Engineer Checklist

  • 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

Copy link

codecov bot commented Feb 26, 2024

Codecov Report

Attention: Patch coverage is 33.33333% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 67.5%. Comparing base (ab836af) to head (7a43c1c).

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff               @@
##             mwcore-dev   #3087     +/-   ##
==============================================
- Coverage          67.6%   67.5%   -0.2%     
+ Complexity          901     900      -1     
==============================================
  Files               165     165             
  Lines              7384    7389      +5     
  Branches           1045    1048      +3     
==============================================
- Hits               4997    4992      -5     
- Misses             1741    1752     +11     
+ Partials            646     645      -1     
Flag Coverage Δ
engine 67.5% <33.3%> (-0.2%) ⬇️

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

Files Coverage Δ
...egister/fhircore/engine/ui/login/LoginViewModel.kt 77.7% <33.3%> (-1.4%) ⬇️

... and 1 file with indirect coverage changes

@calmwalija calmwalija closed this Feb 27, 2024
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.

1 participant