[16.0][MIG] l10n_br_account_payment_brcobranca

[antoniospneto]

Migração do módulo l10n_br_account_payment_brcobranca versão 15.0 para 16.0 Feito a partir da PR: #3224

Depende de:

• [x] https://github.com/OCA/l10n-brazil/pull/3222
• [ ] https://github.com/OCA/l10n-brazil/pull/2865

Edit: Apesar de não depender diretamente do módulo l10n_br_account, quando instalados juntos pode haver conflitos que devem se resolvidos, para facilitar a migração do account, é melhor aguardar o merge lá primeiro.

[rvalyi]

a mudança do TransactionCase ela podia ter sido feita no PR da v15. Mas enfim para ta OK mesmo assim. Valeu demais pelo trabalho!

[rvalyi]

/ocabot migration l10n_br_account_payment_brcobranca

[OCA-git-bot]

The migration issue (#2153) has not been updated to reference the current pull request because a previous pull request (#2967) is not closed. Perhaps you should check that there is no duplicate work. CC @ygcarvalh

[rvalyi]

/ocabot migration l10n_br_account_payment_brcobranca

[rvalyi]

Pessoal, no PR de migracao do l10n_br_account #2865 rolou varios "conflitos" nos testes: tive que desativar uns 2 testes do l10n_br_payment_order, os testes do do l10n_br_cnab_structure e o hook do l10n_br_cnab_structure...

Nisso como a migraçao do l10n_br_account ta impedindo a migraçao de varios outros modulos bem importante e que ja eh bem dificil finalizar a migraçao do l10n_br_account, proponho da gente segurar um pouco o merge desse modulo para nao dar mais tiros no pe ate a gente fazer o merge do l10n_br_account.

cc @renatonlima @antoniospneto @mbcosta

[antoniospneto]

Pessoal, no PR de migracao do l10n_br_account #2865 rolou varios "conflitos" nos testes: tive que desativar uns 2 testes do l10n_br_payment_order, os testes do do l10n_br_cnab_structure e o hook do l10n_br_cnab_structure...

Nisso como a migraçao do l10n_br_account ta impedindo a migraçao de varios outros modulos bem importante e que ja eh bem dificil finalizar a migraçao do l10n_br_account, proponho da gente segurar um pouco o merge desse modulo para nao dar mais tiros no pe ate a gente fazer o merge do l10n_br_account.

cc @renatonlima @antoniospneto @mbcosta

Por mim, tudo bem, vou deixa em rascunho então enquanto aguardamos o l10n_br_account. Estou estudando as mudanças no módulo account nativo, para poder colaborar mais na migração.

[rvalyi]

@antoniospneto eu consegui resolver os problemas de incompatibilidade no l10n_br_account: era o fato de usar testes do country_code com BR em vez de testar a presença de fiscal_operation_id. Deixei ate pronto para ser revisado: https://github.com/OCA/l10n-brazil/pull/2865

Nisso ta tudo bem a gente fazer o merge do l10n_br_account_payment_brcobranca. Pode botar para ready for review de novo?

cc @mbcosta @renatonlima

[rvalyi]

eu ate botei pronto para ser revisado eu mesmo.

[rvalyi]

/ocabot merge nobump

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-3226-by-rvalyi-bump-nobump, awaiting test results.

[OCA-git-bot]

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