josehernandezfintecheandomx commented on code in PR #2442:
URL: https://github.com/apache/fineract/pull/2442#discussion_r932830922
##########
fineract-provider/src/main/java/org/apache/fineract/portfolio/loanaccount/domain/Loan.java:
##########
@@ -292,6 +293,10 @@ public class Loan extends AbstractPersistableCustom {
@Column(name = "loan_product_counter")
private Integer loanProductCounter;
+ @ManyToOne(fetch = FetchType.LAZY)
+ @JoinColumn(name = "delinquency_range_id", nullable = true)
+ private DelinquencyRange delinquencyRange;
Review Comment:
It is to represent the current tag, considering LoanDelinquencyTags could
work for the next scenarios:
1. Loan without delinquency tag because It is active (no delinquency yet)
2. Loan with one or more delinquency tags (history) and you can take the
latest one to know the current delinquency tag,
But In the scenario, after go out of delinquency, when the loan goes back to
active, how you can reflect that? No delinquency
Plus It can help to know quickly If the loan has or not a tag, If It has a
tag, exactly which is it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
