WEB-657: Working Capital Loan Disbursement and Discount fix#3623
WEB-657: Working Capital Loan Disbursement and Discount fix#3623somasorosdpc wants to merge 1 commit into
Conversation
|
Note
|
| Layer / File(s) | Summary |
|---|---|
Discount Fee Type Model src/app/loans/models/loan-transaction-type.model.ts |
LoanTransactionType interface now includes discountFee: boolean field to classify transaction types as discount-fee related. |
Discount Fee Transaction Undo Prevention src/app/loans/loans-view/transactions-tab/transactions-tab.component.ts |
New isDiscountFee() helper detects discount-fee transactions via the flag or transaction-type code matching. allowUndoTransaction() now includes this check to block undo for discount-fee transactions. |
Discount Amount Form Control Initialization src/app/loans/loans-view/loan-account-actions/approve-loan/approve-loan.component.ts, src/app/loans/loans-view/loan-account-actions/disburse/disburse.component.ts |
Both approve-loan and disburse components now initialize discountAmount form controls with values from the backing data object and set disabled state based on the overrideDiscountDisabled flag, replacing prior uninitialized controls. |
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~12 minutes
Possibly related PRs
- openMF/web-app#3447: Updates working-capital
discountAmountreactive form setup in disburse and approve-loan flows.
Suggested reviewers
- alberto-art3ch
- adamsaghy
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
| Check name | Status | Explanation |
|---|---|---|
| Title check | ✅ Passed | The title directly relates to the main changes: initializing and disabling discount fields in approve/disburse loan workflows, and preventing undo of discount transactions for Working Capital loans. |
| Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. |
| Linked Issues check | ✅ Passed | Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | ✅ Passed | Check skipped because no linked issues were found for this pull request. |
| Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. |
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing Touches
🧪 Generate unit tests (beta)
- Create PR with unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands and usage tips.
Description
Working capital loan product
Related issues and discussion
WEB-657
Screenshots, if any
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!
If you have multiple commits please combine them into one commit by squashing them.
Read and understood the contribution guidelines at
web-app/.github/CONTRIBUTING.md.Summary by CodeRabbit
Bug Fixes
New Features