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

Adjust PaymentPlan model to contain informations coming from TP and HS #4372

Draft
wants to merge 137 commits into
base: develop
Choose a base branch
from

Conversation

pavlo-mk
Copy link
Contributor

@pavlo-mk pavlo-mk commented Oct 25, 2024

AB#215785: Adjust PaymentPlan model to contain informations coming from TP and HS
AB#215790: Adjust Payment Module to filter out the Payment Plan which are draft (old Target Population)

@pavlo-mk pavlo-mk added long-term-branch backend Changes in BE code labels Oct 25, 2024
@pavlo-mk pavlo-mk self-assigned this Oct 25, 2024
src/hct_mis_api/apps/payment/celery_tasks.py Outdated Show resolved Hide resolved
src/hct_mis_api/apps/payment/celery_tasks.py Show resolved Hide resolved
src/hct_mis_api/apps/payment/models.py Outdated Show resolved Hide resolved
src/hct_mis_api/apps/payment/models.py Outdated Show resolved Hide resolved
src/hct_mis_api/apps/payment/models.py Outdated Show resolved Hide resolved
Base automatically changed from long-term/remove-tp to develop November 20, 2024 13:16
Copy link

codecov bot commented Nov 22, 2024

Codecov Report

Attention: Patch coverage is 97.78247% with 21 lines in your changes missing coverage. Please review.

Project coverage is 85.01%. Comparing base (9099556) to head (c244601).
Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
src/hct_mis_api/apps/payment/models/payment.py 96.77% 11 Missing ⚠️
...api/apps/payment/services/payment_plan_services.py 98.33% 3 Missing ⚠️
src/hct_mis_api/apps/accountability/schema.py 80.00% 1 Missing ⚠️
...ps/accountability/services/survey_crud_services.py 85.71% 1 Missing ⚠️
src/hct_mis_api/apps/payment/filters.py 97.14% 1 Missing ⚠️
src/hct_mis_api/apps/payment/mutations.py 98.94% 1 Missing ⚠️
src/hct_mis_api/apps/payment/schema.py 98.33% 1 Missing ⚠️
src/hct_mis_api/apps/program/admin.py 0.00% 1 Missing ⚠️
src/hct_mis_api/apps/utils/admin.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #4372      +/-   ##
===========================================
+ Coverage    84.98%   85.01%   +0.02%     
===========================================
  Files          514      514              
  Lines        36269    36278       +9     
===========================================
+ Hits         30824    30841      +17     
+ Misses        5445     5437       -8     
Flag Coverage Δ
e2e 60.44% <67.26%> (-2.02%) ⬇️
unittests 83.30% <97.04%> (+0.07%) ⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Changes in BE code work in progress Change doesn't need to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants