|
For charges being associated with loans, the payment type (Deduct from savings or a regular payment) is currently being captured.
These fields are also mandatory while defining a charge for a savings account. The scope of this task is to simply ignore this field for savings charges (and hide the same from reference UI for savings charges)
Going forward, it might make sense to change this field to a Boolean flag on the database, say "deduct_from_linked_savings" and have the charges API pass a true value for charges to be deducted from savings (default set to false, so that the same need not be passed in from API compulsorily even for loans)
The second bit is not a priority, but making a note of it anyway and tagging Pramod Nuthakki
|
