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

FINERACT-2152: Calculate interest pause on repayment schedule #4244

Conversation

oleksii-novikov-onix
Copy link
Contributor

@oleksii-novikov-onix oleksii-novikov-onix commented Jan 8, 2025

Implemented interest pause for a single period and between two periods.

Checklist

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Write the commit message as per https://github.com/apache/fineract/#pull-requests

  • Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

  • Create/update unit or integration tests for verifying the changes made.

  • Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

  • Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes

  • Submission is not a "code dump". (Large changes can be made "in repository" via a branch. Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

@oleksii-novikov-onix oleksii-novikov-onix force-pushed the FINERACT-2152/calculate-interest-pause-on-repayment-schedule branch 2 times, most recently from 4e2abe8 to 421c8f2 Compare January 15, 2025 12:59
@oleksii-novikov-onix oleksii-novikov-onix force-pushed the FINERACT-2152/calculate-interest-pause-on-repayment-schedule branch 3 times, most recently from 07103c5 to 4e87585 Compare January 20, 2025 11:59
@oleksii-novikov-onix oleksii-novikov-onix marked this pull request as ready for review January 20, 2025 13:56
@adamsaghy
Copy link
Contributor

@oleksii-novikov-onix Kindly resolve the conflicts!

@oleksii-novikov-onix oleksii-novikov-onix force-pushed the FINERACT-2152/calculate-interest-pause-on-repayment-schedule branch from 4e87585 to 869e1ba Compare January 21, 2025 08:45
@oleksii-novikov-onix oleksii-novikov-onix force-pushed the FINERACT-2152/calculate-interest-pause-on-repayment-schedule branch from 41ac30e to 0a545c0 Compare January 21, 2025 15:51
@oleksii-novikov-onix oleksii-novikov-onix marked this pull request as draft January 21, 2025 16:08
@oleksii-novikov-onix oleksii-novikov-onix force-pushed the FINERACT-2152/calculate-interest-pause-on-repayment-schedule branch 2 times, most recently from 7f0a743 to 6204bd2 Compare January 23, 2025 13:03
@oleksii-novikov-onix oleksii-novikov-onix marked this pull request as ready for review January 23, 2025 14:46
@oleksii-novikov-onix oleksii-novikov-onix force-pushed the FINERACT-2152/calculate-interest-pause-on-repayment-schedule branch from ef41ed0 to 03a2769 Compare January 23, 2025 14:48
Copy link
Contributor

@janez89 janez89 left a comment

Choose a reason for hiding this comment

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

Approved after few common review session

@oleksii-novikov-onix oleksii-novikov-onix force-pushed the FINERACT-2152/calculate-interest-pause-on-repayment-schedule branch from 03a2769 to c1fee3e Compare January 23, 2025 16:15
@adamsaghy adamsaghy merged commit af37e07 into apache:develop Jan 24, 2025
10 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