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.

Sign up for GitHub

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

Jump to bottom

fix: incorrect cost_center in ar/ap #43174

Draft

ljain112 wants to merge 1 commit into frappe:develop from ljain112:fix-ar-ap

Contributor

ljain112 commented Sep 11, 2024

If there is a journal entry with the same accounts but a different cost centre Accounts Receivable shows an incorrect cost center.

Support Issue: https://support.frappe.io/app/hd-ticket/21671


          fix: incorrect cost_center in ar/ap

e922445

ljain112 requested review from deepeshgarg007 and ruthra-kumar as code owners

September 11, 2024 12:34

github-actions bot added the needs-tests label

ljain112 marked this pull request as draft

September 11, 2024 13:30

ruthra-kumar self-assigned this

ruthra-kumar removed the request for review from deepeshgarg007

September 17, 2024 05:09

Member

ruthra-kumar commented Sep 17, 2024

@ljain112
Let's not add cost_center to the key variable. Accounting Dimension is not a mandatory field to link Invoices with Payments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels