Skip to content

Conversation

@marcelsavegnago
Copy link
Member

No description provided.

weblate and others added 30 commits April 28, 2025 16:43
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_account
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_account/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_account
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_account/
Currently translated at 97.9% (141 of 144 strings)

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_account
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_account/pt_BR/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_account
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_account/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_account
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_account/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-brazil-14.0/l10n-brazil-14.0-l10n_br_account
Translate-URL: https://translation.odoo-community.org/projects/l10n-brazil-14-0/l10n-brazil-14-0-l10n_br_account/
use the native Odoo account invoice tests to ensure
l10n_br_account don't destroy native accounting features
for non Brazilian companies.
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_account branch 2 times, most recently from 2caa270 to 532dc6a Compare April 30, 2025 21:01
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_account branch from 11434b8 to 28c0201 Compare April 30, 2025 21:18
@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_account branch 3 times, most recently from 05bd839 to bd084f2 Compare May 7, 2025 01:52
@marcelsavegnago
Copy link
Member Author

marcelsavegnago commented May 14, 2025

Peguei uma questão durante os testes.

No modulo l10n_br_base os dados da empresa principal é alterado inclusive mudando a moeda para BRL. Porém, na versão 17 tem uma linha no metodo setupclass do arquivo common.py (pasta tests) que força a mudança da moeda para USD e no caso dá erro porque esta empresa já tem lançamentos de diario neste momento.

image

https://github.com/odoo/odoo/blob/05734a7ae5f6c6ec272178a1ab364db3c88af64d/addons/account/tests/common.py#L50

odoo/odoo@b1537a0

cc @rvalyi @antoniospneto @kaynnan @mileo

@antoniospneto
Copy link
Contributor

@marcelsavegnago talvez uma alternativa é a gente usar as empresas simples nacional e lucro presumido para os testes e demonstrações da nossa localização, e deixar de usar a empresa principal.

@rvalyi
Copy link
Member

rvalyi commented May 14, 2025

@marcelsavegnago talvez uma alternativa é a gente usar as empresas simples nacional e lucro presumido para os testes e demonstrações da nossa localização, e deixar de usar a empresa principal.

Seria bem razoável sim. Mas ai seria interessante começar a fazer isso na 16.0 para não começar a divergir muito nos testes, senão vai ser foda manter viu... Pros testes que não mexem com valores como CRM, hr, zip ou coisa do tipo ta de boa continuar a usar o main_company.

@rvalyi
Copy link
Member

rvalyi commented May 14, 2025

@marcelsavegnago @antoniospneto outra saida que pode ser mais simples a curto prazo seria habilitar de fazer o write de volta para a moeda base.USD aqui no res_company onde ja fizemos uma gambiarra pro base.BRL:

https://github.com/OCA/l10n-brazil/blob/16.0/l10n_br_base/models/res_company.py#L168

Mas assim a longo prazo vai ser melhor focar os testes nas empresas simples e lucro presumido sim.

@marcelsavegnago marcelsavegnago force-pushed the 17.0-mig-l10n_br_account branch from bd084f2 to c157d3f Compare May 19, 2025 14:22
@github-actions
Copy link

github-actions bot commented Dec 7, 2025

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Dec 7, 2025
@rvalyi
Copy link
Member

rvalyi commented Jan 6, 2026

substituído por #4339
Eu consegui reaproveitar varios commits, alguns eu amendei...

@rvalyi rvalyi closed this Jan 7, 2026
@marcelsavegnago marcelsavegnago deleted the 17.0-mig-l10n_br_account branch January 8, 2026 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

17.0 migration stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.