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

[12.0] Brazilian Chart of Account #34160

Closed
wants to merge 10 commits into from

Conversation

renatonlima
Copy link
Contributor

@renatonlima renatonlima commented Jun 15, 2019

Hi guys,

Today we can't use l10n_br chart of account since 9.0 when there were many changes in account, I think the module has been migrated automatically and some information does not represent the configuration of the data in Brazil, for example the creation of account.tax.tag is not correct (it is not necessary to create tags for each percentage of taxes) Also the account.tag is not correct (account.group should be created to create the accounting account hierarchy), now we are migrating Brazilian localization OCA/l10n-brazil to 12.0 and I updated l10n_br including:

  • Removed creating fields in core objects account.tax.template and account.tax and its views;
  • Update tax group;
  • Update tax template (Need to update tax percentages);
  • Update account.group to implement hierarchy in accounting reports;
  • Update chart of account according Brazilian Federal Accounting Council Resolution No. 1.418/12

I will do one PR in master branch to next version, but it would be great to have the chart of account updated in version 12.0

@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Jun 15, 2019
@jco-odoo jco-odoo self-requested a review June 19, 2019 08:45
@jco-odoo jco-odoo added the Localization about adapting the content to a specify country (usually chart of accounts) label Jun 19, 2019
@jco-odoo jco-odoo self-assigned this Jun 19, 2019
Copy link
Contributor

@jco-odoo jco-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can certainly update the CoA and thanks for wanting to contribute this, but we should keep the tax tags and make sure they correspond to the VAT report.

I don't know about the quality of those OCA modules and maybe we could rather suggest to contact a partner if they are interested in more functionality.

addons/l10n_br/data/account_data.xml Show resolved Hide resolved
addons/l10n_br/models/account.py Show resolved Hide resolved
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed CI 🤖 Robodoo has seen passing statuses labels Jun 24, 2019
@renatonlima
Copy link
Contributor Author

Hi @jco-odoo,

Thank you for your review, I resolved your revisions in this PR, Can you review?

@renatonlima
Copy link
Contributor Author

I also updated my PR in master branch #34163

@renatonlima
Copy link
Contributor Author

ping @jco-odoo

@C3POdoo C3POdoo added the CLA Contributor License Agreement label Dec 20, 2019
@renatonlima
Copy link
Contributor Author

@jco-odoo I resolved branch file conflict

@C3POdoo
Copy link
Contributor

C3POdoo commented Apr 22, 2022

Dear @renatonlima,

Thank you for your contribution but the version 12.0 is no longer supported.
We only support the last 3 stable versions so no longer accepts patches into this branch.

We apology if we could not look at your request in time.
If the contribution still makes sense for the upper version, please let us know and do not hesitate to recreate one for the recent versions. We will try to check it as soon as possible.

This is an automated message.

@C3POdoo C3POdoo closed this Apr 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses CLA Contributor License Agreement Localization about adapting the content to a specify country (usually chart of accounts)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants