Skip to content
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

[FIX] ACH Payment Unbalanced Entry #10

Open
wants to merge 22 commits into
base: 14.0-MIG-account_banking_ach_discount
Choose a base branch
from

Conversation

atchuthan
Copy link

@atchuthan atchuthan commented Sep 1, 2021

(Internal Notes: [ID:20716] Issue 1057: ACH Payment Unbalanced Entry)
depends on OCA#73
cc: @SodexisTeam

@atchuthan atchuthan force-pushed the 14.0-MIG-account_banking_ach_discount-unbalanced-entry branch from 223c2c3 to 2be02ad Compare September 3, 2021 10:17
@Murtaza-OSI Murtaza-OSI force-pushed the 14.0-MIG-account_banking_ach_discount branch from d77080d to 8da666a Compare September 30, 2021 06:42
@bodedra
Copy link

bodedra commented Oct 13, 2021

@atchuthan Thanks for your contribution.

I have reviewed your fix and it will discontinue to create discount journal items. That was not the original intent of the module.

We want to record a discount amount in a separate account (which is configured at payment term line level) for an income/expense accounting report.

There should be bugs somewhere else.

I leave this PR open.

Thank you.

@atchuthan atchuthan force-pushed the 14.0-MIG-account_banking_ach_discount-unbalanced-entry branch from 7c519f5 to b2fa98f Compare April 13, 2022 13:54
@dreispt dreispt force-pushed the 14.0-MIG-account_banking_ach_discount branch 2 times, most recently from 7119312 to e00eee6 Compare November 15, 2022 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

8 participants