Skip to content

[16.0][FIX] l10n_br_coa_generic: accounts update #3775

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

Draft
wants to merge 2 commits into
base: 16.0
Choose a base branch
from

Conversation

rvalyi
Copy link
Member

@rvalyi rvalyi commented May 6, 2025

Na Akretion nunca usamos esse plano de Conta em produção (usamos os próprios planos dos clientes quando as empresas são do regime normal), então com pé atrás eu joguei no AI Gemini para checar um pouco e ela fez essas muitas sugestões... Provavelmente que nem tudo que foi sugerido ta certo, mas é provável que teria sim que ajustar muita coisa... Seria bom avaliar sugestão por sugestão...

Key High-Confidence Changes Made:

    Critical account_type Corrections:

        Cost/Inventory Transfer Accounts (e.g., 1.1.3.1.19 (-) Custo das Mercadorias Vendidas, 1.1.3.3.19 (-) Transferência para Consumo Matérias Primas): Changed from asset_current to expense_direct_cost (or expense for "Uso e Consumo"). This is crucial for correct P&L reporting.

        Accumulated Depreciation/Amortization (Balance Sheet) (e.g., 1.2.8.9.01 (-)): Changed from expense_depreciation to asset_fixed (as they are contra-asset accounts).

        Compensation Accounts (Groups 1.9.x.x.xx and 2.9.x.x.xx): Changed from various incorrect types (like liability_non_current) to off_balance.

        Production Cost Accounts (Group 3.x.x.x.xx): Changed from liability_non_current to expense_direct_cost.

        Production Transfer Accounts (Group 4.x.x.x.xx): Changed from liability_non_current to expense_direct_cost (as they are typically contra-cost or transfer accounts).

        Expense Accounts (Many in Group 5.x.x.x.xx): Corrected from income_other to appropriate expense types (expense or expense_depreciation).

        Revenue Deduction / Contra-Revenue Accounts (Group 6.1.1.2.xx, 6.1.1.3.xx): Changed from income_other to income (these accounts reduce gross income).

        Equity Reserve Accounts (e.g., 2.4.2.1.01 Reserva de Incentivos Fiscais): Changed from liability_non_current to equity.

        Apuração Accounts (Group 7.x.x.x.xx): Changed from liability_non_current to equity.

        Crucial: coa_generic_712101 (Resultado Final de Exercício): Ensured this is equity_unaffected for correct Odoo year-end closing. (Assuming this line was intended to be in your full CSV).

    reconcile Flag Adjustments:

        coa_generic_112201 (-) Duplicatas Descontadas: Changed reconcile from 1 to 0.

        coa_generic_113101 / coa_generic_119101 (Estoque Inicial Mercadorias): Changed reconcile from 1 to 0.

        coa_generic_119001/coa_generic_119002 (Estoque Intermediário): Changed account_type to asset_current and reconcile to 0.

    Other account_type Refinements:

        Adiantamentos (Advances) like 1.1.4.2.xx and 1.1.4.3.xx: Changed from asset_current to asset_prepayments.

        A Apropriar (Prepaids) like 1.1.5.1.xx: Changed from asset_current to asset_prepayments.


Important Considerations After Import:

    Duplicated Inventory Blocks (1.1.3.x vs 1.1.9.x): The CSV contains highly similar, almost duplicated blocks of inventory accounts (e.g., 1.1.3.1.01 and 1.1.9.1.01 are both "Estoque Inicial Mercadorias"). I've corrected both based on the types provided, but you should review if this duplication is intentional for your setup or if one block should be removed to avoid confusion. I've added "(Duplicado?)" to the notes for the 1.1.9.x.xx block for your review.

    Periodic vs. Perpetual Inventory: This chart of accounts seems designed for a very detailed periodic inventory system (with accounts for "Compras," "Fretes s/ Compras," "ICMS s/ Compras," etc., and then a "(-) Custo das Mercadorias Vendidas" as a contra-asset/cost account). Odoo's default is perpetual inventory with automated COGS postings. You'll need to ensure your Odoo product categories and journal configurations align with how you intend to use these accounts, or simplify this section if you plan to use Odoo's automated COGS.

    Testing: Thoroughly test journal entries, P&L, and Balance Sheet reports after importing to ensure accounts behave as expected with these new types.

@OCA-git-bot
Copy link
Contributor

Hi @mileo,
some modules you are maintaining are being modified, check this out!

@rvalyi rvalyi marked this pull request as draft May 6, 2025 15:14
@rvalyi rvalyi changed the title [REF] l10n_br_coa_generic: accounts update [16.0][REF] l10n_br_coa_generic: accounts update May 6, 2025
@rvalyi rvalyi force-pushed the 16.0-cfc-account-updates branch from 3a18c5a to 65aca8f Compare May 6, 2025 16:28
@rvalyi rvalyi changed the title [16.0][REF] l10n_br_coa_generic: accounts update [16.0][FIX] l10n_br_coa_generic: accounts update May 6, 2025
@antoniospneto antoniospneto added this to the 16.0 milestone May 6, 2025
@marcelsavegnago
Copy link
Member

está pronta para revisao ?

Comment on lines +91 to +94
coa_generic_119111,1.1.9.1.11,(-) ICMS sobre Compras Mercadorias (Duplicado?),,asset_current,0,l10n_br_coa_generic_template
coa_generic_119112,1.1.9.1.12,(-) COFINS sobre Compras Mercadorias (Duplicado?),,asset_current,0,l10n_br_coa_generic_template
coa_generic_119113,1.1.9.1.13,(-) PIS sobre Compras Mercadorias (Duplicado?),,asset_current,0,l10n_br_coa_generic_template
coa_generic_119119,1.1.9.1.19,(-) Custo das Mercadorias Vendidas (Duplicado?),,expense_direct_cost,0,l10n_br_coa_generic_template
Copy link
Contributor

@antoniospneto antoniospneto May 8, 2025

Choose a reason for hiding this comment

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

esses "(duplicados?)" estão entranhos não?

@rvalyi
Copy link
Member Author

rvalyi commented May 8, 2025

está pronta para revisao ?

então, como vc queria migrar para a v17 de um lado e do outro lado tem o @mileo que pretende ser mantenedor do modulo, dado as varias sugestões da AI Gemini, eu acharia bom dar uma revisada ainda que superficial nesse PR antes de jogar o plano para a v17 com formato de csv totalmente diferente (ou seja impossivel de sincronizar automaticamente depois).

Por outro lado, não é prioridade da Akretion acertar esse plano 100%, nem migrar para a 17 agora e por fim a gente nem confiaria no IA 100% para fazer esse tipo de alteração.

Então minha sugestão é que esse PR serve de base para quem pretende migrar logo para a 17.0+ ou que se pretende mantenedor do modulo.

Nisso eu acho que o PR ja pode servir de base para as pessoas opinar ou não sobre essas alterações... Novamente eu diria que não precisa procurar acertar 100% agora, mas com certeza naquilo tudo tem correções importante a serem feitas...

cc @OCA/local-brazil-maintainers @DiegoParadeda

@marcelsavegnago
Copy link
Member

marcelsavegnago commented May 8, 2025

está pronta para revisao ?

então, como vc queria migrar para a v17 de um lado e do outro lado tem o @mileo que pretende ser mantenedor do modulo, dado as varias sugestões da AI Gemini, eu acharia bom dar uma revisada ainda que superficial nesse PR antes de jogar o plano para a v17 com formato de csv totalmente diferente (ou seja impossivel de sincronizar automaticamente depois).

Por outro lado, não é prioridade da Akretion acertar esse plano 100%, nem migrar para a 17 agora e por fim a gente nem confiaria no IA 100% para fazer esse tipo de alteração.

Então minha sugestão é que esse PR serve de base para quem pretende migrar logo para a 17.0+ ou que se pretende mantenedor do modulo.

Nisso eu acho que o PR ja pode servir de base para as pessoas opinar ou não sobre essas alterações... Novamente eu diria que não precisa procurar acertar 100% agora, mas com certeza naquilo tudo tem correções importante a serem feitas...

cc @OCA/local-brazil-maintainers @DiegoParadeda

Concordo totalmente que cabe antes de qq coisa em relação a 17.. perguntei se estava pronta para revisão porque está marcada como rascunho.

@rvalyi
Copy link
Member Author

rvalyi commented May 8, 2025

@marcelsavegnago deixei de rascunho porque é quase que certeza que tem coisas zoada nisso que o AI reportou... Se tivesse um estagio "ta meio pronto" eu teria usado ;-) Mas eu diria que podem revisar sim.

Ai de acordo com as revisões:

  • ou eu vou alterando se for mudanças razoáveis
  • ou alguém (tipo que tem fogo no rabo para migrar ou que quer bancar o mantenedor), faz novos PRs ou sub-PRs...

Isso fica a definir...

@marcelsavegnago
Copy link
Member

@marcelsavegnago deixei de rascunho porque é quase que certeza que tem coisas zoada nisso que o AI reportou... Se tivesse um estagio "ta meio pronto" eu teria usado ;-) Mas eu diria que podem revisar sim.

Ai de acordo com as revisões:

  • ou eu vou alterando se for mudanças razoáveis
  • ou alguém (tipo que tem fogo no rabo para migrar ou que quer bancar o mantenedor), faz novos PRs ou sub-PRs...

Isso fica a definir...

opa.. blzzz. vlw

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants