alyf-de / banking

An Open Banking Integration with ERPNext
GNU General Public License v3.0
37 stars 18 forks source link

Paying the last invoice with full amount even the created payment entry in Bank transaction has less amount #85

Closed aa-prashant closed 7 months ago

aa-prashant commented 8 months ago
Bank Transaction

The Amount of Payment Entry showing in the bank transaction is $20.07 - Because that is the remaining amount

Actual

The Actual Amount paid in the payment entry is $335 (The actual Amount of the Invoice), It has paid of the complete invoice, but it should have been 20.07 and partially paying the invoice.

This is a terrifying issue, and I request you to solve it quickly. I am also requesting you to provide a script which can diagnose and track all the payments/invoices which are fully paid by these issues. (We can have a script which tracks payment entries in the bank transaction child table and the respective payment entry document and finds out if there is a difference in amount.)

aa-prashant commented 8 months ago

Here is the full screenshot, Showing the payment entry number

Screenshot 2023-11-17 at 7 04 27 PM
barredterra commented 8 months ago

Hi, @aa-prashant kindly share the version of the banking app you're using. (We've already resolved very similar issues, so you might still experience them if you're on an older version.)

aa-prashant commented 8 months ago

Hi, @aa-prashant kindly share the version of the banking app you're using. (We've already resolved very similar issues, so you might still experience them if you're on an older version.)

ALYF Banking: v0.0.9 (version-14)

marination commented 8 months ago

this is in regards to reco of an unpaid SI against a bank transaction with lesser amount to allocate right ? @barredterra shouldn't that be fixed by https://github.com/alyf-de/banking/blob/171895a31c762bcba75d835efdf7eac0a8f0e1ca/banking/overrides/bank_transaction.py#L50

marination commented 7 months ago

@aa-prashant the issue has been fixed and released. You could update your banking instance.

aa-prashant commented 7 months ago

@aa-prashant the issue has been fixed and released. You could update your banking instance.

Thank you so much to the team, will try and get the feedback here.

Thank you