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

[ENH] Add Reference Chart of Accounts template (RGS). #48

Merged
merged 3 commits into from
Sep 20, 2016

Conversation

NL66278
Copy link
Contributor

@NL66278 NL66278 commented Jun 16, 2016

Give users the option to use the reference chart of accounts (Referentie Grootboek Schema).

See: http://www.referentiegrootboekschema.nl/home

@lfreeke
Copy link
Contributor

lfreeke commented Jun 16, 2016

@NL66278 please check travis error

@NL66278 NL66278 force-pushed the 8.0-account-chart-rgs branch 3 times, most recently from 63edc17 to 1bf5131 Compare June 16, 2016 11:35
@NL66278
Copy link
Contributor Author

NL66278 commented Jun 16, 2016

@lfreeke Travis errors solved. @StefanRijnhart As you requested a module in the l10n-netherlands repo.

@StefanRijnhart
Copy link
Member

Thanks! Some initial feedback:

Can you fill the properties (such as property_account_receivable) which are defined in the regular Dutch chart of accounts?
Some accounts seem to have the wrong category. This is something I am not an expert in either, but I think 'Voorziening dubieuze debiteuren' is not a debtor account for instance, just like 'Vooruitbetalingen op bestelling' is not a creditor account. Maybe we should have the assignment of categories checked by an accountant.
In the account mapping in the Niet EU Landen fiscal position, there are double mappings. For instance, 8001010 maps to multiple accounts.

@NL66278
Copy link
Contributor Author

NL66278 commented Jun 20, 2016

@StefanRijnhart Hello I corrected the debtor and creditor acounts.

There is no work to be done on properties. I think they are created using the information from the chart template (in account_chart_template.xml), this is no different from the original dutch chart of account.
I cannot find the double mappings in the fiscal position. And I do not use the names like 8001010 to refer to accounts, but the xml_id's. Where exactly did you see this?

@hbrunn
Copy link
Member

hbrunn commented Jun 22, 2016

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

hbrunn commented Jul 25, 2016

👍

@lfreeke
Copy link
Contributor

lfreeke commented Sep 15, 2016

👍 (test) At first sight it all seems to be working.

@hbrunn hbrunn merged commit 2844555 into OCA:8.0 Sep 20, 2016
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

4 participants