-
Notifications
You must be signed in to change notification settings - Fork 2.2k
FINERACT-2354: Introduce reaging preview API #5063
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-2354: Introduce reaging preview API #5063
Conversation
8d5cc27 to
61b0528
Compare
|
@oleksii-novikov-onix Please rebase. |
61b0528 to
377db88
Compare
|
377db88 to
ff13fb2
Compare
cbb30b1 to
3a8af86
Compare
...oanaccount/domain/transactionprocessor/impl/AdvancedPaymentScheduleTransactionProcessor.java
Outdated
Show resolved
Hide resolved
...oanaccount/domain/transactionprocessor/impl/AdvancedPaymentScheduleTransactionProcessor.java
Outdated
Show resolved
Hide resolved
...src/main/java/org/apache/fineract/portfolio/loanaccount/api/LoanTransactionsApiResource.java
Outdated
Show resolved
Hide resolved
...src/main/java/org/apache/fineract/portfolio/loanaccount/api/LoanTransactionsApiResource.java
Outdated
Show resolved
Hide resolved
.../main/java/org/apache/fineract/portfolio/loanaccount/service/reaging/LoanReAgingService.java
Outdated
Show resolved
Hide resolved
adamsaghy
left a comment
There was a problem hiding this 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!
3a8af86 to
c3b04b9
Compare
5d47942 to
cc7407d
Compare
d16b458 to
568e2f3
Compare
568e2f3 to
2055a0e
Compare
|
@oleksii-novikov-onix Please rebase |
2055a0e to
5179b33
Compare
5179b33 to
cb2a09a
Compare
cb2a09a to
0dd13c8
Compare
| | 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" |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed, thanks
adamsaghy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
0dd13c8 to
011006b
Compare
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!
FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.