OCA / l10n-brazil

Localização brasileira oficial do Odoo.
https://odoo-community.org/psc-teams/brazil-66
GNU Affero General Public License v3.0
236 stars 244 forks source link

[16.0][MIG][l10n_br_stock_account] #3275

Closed rvalyi closed 2 weeks ago

rvalyi commented 1 month ago

migraçao para a v16 em cima da migraçao v15

depende de https://github.com/OCA/l10n-brazil/pull/2865

instala mas ainda nao passa os testes...

rvalyi commented 1 month ago

/ocabot migration l10n_br_stock_account

rvalyi commented 3 weeks ago

@renatonlima @marcelsavegnago @antoniospneto @mbcosta a migraçao ficou pronta para review, passou tudo certinho.

antoniospneto commented 3 weeks ago

@renatonlima @marcelsavegnago @antoniospneto @mbcosta a migraçao ficou pronta para review, passou tudo certinho.

Na v15 foi desabilitado alguns testes, esses será que não dá pra habilitar pra ver se agora passa? ou fica para resolver posteriormente?

rvalyi commented 3 weeks ago

hum ok, vou olhar isso, valeu por notar.

rvalyi commented 2 weeks ago

@antoniospneto realmente eu tinha desabilitado alguns testes na época que ainda tinha bastante problemas com a migração do l10n_br_account. Mas agora botei os testes originais de volta e passou tudo. Tb precisei de incluir o commit de #3198 para funcionar.

inclui todos os últimos commits da 14.0 no l10n_br_stock_account tb.

Cc @renatonlima @marcelsavegnago @mbcosta

renatonlima commented 2 weeks ago

/ocabot merge nobump

OCA-git-bot commented 2 weeks ago

What a great day to merge this nice PR. Let's do it! Prepared branch 16.0-ocabot-merge-pr-3275-by-renatonlima-bump-nobump, awaiting test results.

OCA-git-bot commented 2 weeks ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

OCA-git-bot commented 2 weeks ago

Congratulations, your PR was merged at e8b02263719e0fb75334cd8935491b8ebbffb005. Thanks a lot for contributing to OCA. ❤️