Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@oleksii-novikov-onix
Copy link
Contributor

Description

Describe the changes made and why they were made.

Ignore if these details are present on the associated Apache Fineract JIRA ticket.

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-2354/introduce-loan-reaging-preview branch 3 times, most recently from 8d5cc27 to 61b0528 Compare October 6, 2025 06:23
@adamsaghy
Copy link
Contributor

@oleksii-novikov-onix Please rebase.

@oleksii-novikov-onix oleksii-novikov-onix force-pushed the FINERACT-2354/introduce-loan-reaging-preview branch from 61b0528 to 377db88 Compare October 15, 2025 11:27
@oleksii-novikov-onix
Copy link
Contributor Author

@oleksii-novikov-onix Please rebase.

@adamsaghy @MarianaDmytrivBinariks Done

@oleksii-novikov-onix oleksii-novikov-onix force-pushed the FINERACT-2354/introduce-loan-reaging-preview branch from 377db88 to ff13fb2 Compare October 23, 2025 11:41
@MarianaDmytrivBinariks MarianaDmytrivBinariks force-pushed the FINERACT-2354/introduce-loan-reaging-preview branch 2 times, most recently from cbb30b1 to 3a8af86 Compare October 24, 2025 10:54
@oleksii-novikov-onix oleksii-novikov-onix marked this pull request as ready for review October 27, 2025 08:53
Copy link
Contributor

@adamsaghy adamsaghy left a comment

Choose a reason for hiding this comment

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

Please kindly see my review!

@MarianaDmytrivBinariks MarianaDmytrivBinariks force-pushed the FINERACT-2354/introduce-loan-reaging-preview branch from 3a8af86 to c3b04b9 Compare October 27, 2025 09:44
@oleksii-novikov-onix oleksii-novikov-onix force-pushed the FINERACT-2354/introduce-loan-reaging-preview branch 2 times, most recently from 5d47942 to cc7407d Compare October 28, 2025 09:48
@oleksii-novikov-onix oleksii-novikov-onix force-pushed the FINERACT-2354/introduce-loan-reaging-preview branch 4 times, most recently from d16b458 to 568e2f3 Compare November 4, 2025 22:48
@oleksii-novikov-onix oleksii-novikov-onix marked this pull request as draft November 6, 2025 13:48
@oleksii-novikov-onix oleksii-novikov-onix force-pushed the FINERACT-2354/introduce-loan-reaging-preview branch from 568e2f3 to 2055a0e Compare November 7, 2025 10:14
@adamsaghy
Copy link
Contributor

@oleksii-novikov-onix Please rebase

@oleksii-novikov-onix oleksii-novikov-onix force-pushed the FINERACT-2354/introduce-loan-reaging-preview branch from 2055a0e to 5179b33 Compare November 7, 2025 11:27
@adamsaghy adamsaghy marked this pull request as ready for review November 7, 2025 11:29
@oleksii-novikov-onix oleksii-novikov-onix force-pushed the FINERACT-2354/introduce-loan-reaging-preview branch from 5179b33 to cb2a09a Compare November 7, 2025 11:31
@MarianaDmytrivBinariks MarianaDmytrivBinariks force-pushed the FINERACT-2354/introduce-loan-reaging-preview branch from cb2a09a to 0dd13c8 Compare November 7, 2025 17:41
| Principal due | Interest | Fees | Penalties | Due | Paid | In advance | Late | Outstanding |
| 1000.0 | 0.0 | 0.0 | 0.0 | 1000.0 | 0.0 | 0.0 | 0.0 | 1000.0 |

# When Loan Pay-off is made on "15 April 2025"
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe it can be uncommented, or am I missing something?

Copy link
Contributor

Choose a reason for hiding this comment

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

removed, thanks

Copy link
Contributor

@adamsaghy adamsaghy left a comment

Choose a reason for hiding this comment

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

LGTM

@MarianaDmytrivBinariks MarianaDmytrivBinariks force-pushed the FINERACT-2354/introduce-loan-reaging-preview branch from 0dd13c8 to 011006b Compare November 11, 2025 10:20
@adamsaghy adamsaghy merged commit 01ca515 into apache:develop Nov 11, 2025
34 checks passed
@adamsaghy adamsaghy deleted the FINERACT-2354/introduce-loan-reaging-preview branch November 11, 2025 11:04
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.

4 participants