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

feat: temporary against account #24086

Merged
merged 6 commits into from
Jan 25, 2021
Merged

feat: temporary against account #24086

merged 6 commits into from
Jan 25, 2021

Conversation

barredterra
Copy link
Collaborator

@barredterra barredterra commented Dec 8, 2020

In DATEV, the against account has a meaning. It tries to fit everything into one record:

Account Against Account Amount
Revenue (19% VAT) Customer € 119.00

VAT gets deducted and booked to different accounts automatically. Receivables will be tracked, etc. This approach comes to its limitations when there are multiple tax rates, advance payments, etc. on one invoice.

ERPNext however, books like this:

Debit Credit Account
€ 119.00 Debtors
€ 19.00 VAT 19%
€ 100.00 Revenue

This is much more flexible, but the against account has no meaning. We just debit or credit individual accounts.

Previously we tried to guess the against account for the DATEV Export. This was achieved by using one payable/receivable account per party, merging all those tables and coalescing through the values.

From discussion on the forum (german) it turned out that a better strategy would be to use a temporary against account for the whole export. In the external system, all GL Entries will be made against this temporary account and it's balance will always be zero. The column "Belegfeld 1" contains the voucher number, tying multiple rows together into one transaction.

Pros:

  • Entries are identical to ERPNext (because in ERPNext the against account has no meaning).
  • Less code, less errors.
  • No more dedicated accounts for parties, meaning less work in daily business.

Cons:

  • Automatic processing in DATEV will not be possible, meaning more work for the tax consultant.

DATEV Settings

datev_settings

Full Export

full_export

Payment Entry only

payment_entry

Stock Entry only

stock_entry

Docs: frappe/erpnext_documentation#221

@barredterra
Copy link
Collaborator Author

@nabinhait anything I can do to get this merged?

@nabinhait nabinhait merged commit 1991ba7 into develop Jan 25, 2021
@barredterra barredterra deleted the temp_against_account branch January 25, 2021 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants