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

Added module l10n_nl_coa: new NL chart of accounts #47

Closed
wants to merge 1 commit into from
Closed

Added module l10n_nl_coa: new NL chart of accounts #47

wants to merge 1 commit into from

Conversation

astirpe
Copy link
Member

@astirpe astirpe commented Jun 10, 2016

This module provides a more generic and complete chart of accounts, in conjunction with a overall review of the module. New taxes for service (diensten) are added in the tax template.
The compatibility with the actual NL Vat Statements report is kept.

@NL66278
Copy link
Contributor

NL66278 commented Jun 10, 2016

I would request to split the module in two. One for things like tax tags and fiscal positions that are the same, no matter what kind of chart of accounts you use. This could be part of the l10n_nl module were originally proposed. Then this module can add the chart of accounts as proposed here, and others (I volunteer) could propose a module for the chart of accounts according to the reference scheme. The user could then either install the one, or the other module (or even both in a multiple company scenario with different accounting needs).

@coveralls
Copy link

coveralls commented Jun 10, 2016

Coverage Status

Coverage remained the same at 39.286% when pulling 9e2b50c on onesteinbv:90_l10n_nl_coa into 89fcc12 on OCA:9.0.

@StefanRijnhart
Copy link
Member

Agree with @NL66278 that the improvements to the tax configuration can be submitted in odoo/odoo#12351. For a new set of accounts, I remain that RSG is the way forward, as proposed now in #48

@hbrunn hbrunn added this to the 9.0 milestone Jun 22, 2016
@astirpe
Copy link
Member Author

astirpe commented Oct 14, 2016

This proposed module was meant to be a replacement of the standard localization module l10n_nl.

An initial PR (odoo/odoo#12351) was opened towards Odoo V9.
Odoo SA accepted that proposed PR but decided to apply the replacement of module l10n_nl starting from Odoo V10.

So this actual PR could be considered as the back-porting to V9 of the functionality provided with l10n_nl (Odoo V10).

However, to avoid confusion and strange impacts that would occur in the future in case someone uses this module, I close this PR.
For every clarification, don't hesitate to contact me.

@astirpe astirpe closed this Oct 14, 2016
@astirpe astirpe deleted the 90_l10n_nl_coa branch October 14, 2016 10:34
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

5 participants