[16.0][MIG][l10n_br_account] #2865
Conversation
rvalyi
force-pushed
the
16.0-mig-l10n_br_account
branch
6 times, most recently
from
576f132
to
3511a81
Compare
rvalyi
force-pushed
the
16.0-mig-l10n_br_account
branch
2 times, most recently
from
1ad7a3f
to
abfa004
Compare
|
/ocabot migration l10n_br_account |
rvalyi
force-pushed
the
16.0-mig-l10n_br_account
branch
from
abfa004
to
75ca42b
Compare
|
dei um rebase para ficar mais facil testar esse PR |
marcelsavegnago
requested review from
douglascstd,
kaynnan,
WesleyOliveira98,
antoniospneto,
vanderleiromera and
felipemotter
|
pessoal é bom testar o PR, porem vale lembrar que o metodo _compute_amounts não foi migrado ainda assim que eu expliquei em #2865 (comment) |
é provável que sem terminar de migrar o _compute_amounts muitos tests não vão passar nos outros módulos, mas da para começar a ajeitar os outros PRs isso sim... |
|
@rvalyi róla um rebase ? |
rvalyi
force-pushed
the
16.0-mig-l10n_br_account
branch
from
75ca42b
to
bdb10b0
Compare
feito. |
rvalyi
force-pushed
the
16.0-mig-l10n_br_account
branch
from
ae93e04
to
2147660
Compare
|
pessoal, dei um rebase so para mostrar que continuiu tudo bem com o modulo l10n_br_account_payment_brcobranca que teve merge de manha. |
|
Bora mesclar essssa p0rr@ :D |
|
@antoniospneto bora ? |
|
@douglascstd bora ? |
|
Eu não cheguei analisar ainda as últimas alterações, mas concordo em fazer o merge agora. |
|
Bora fazer o merge |
|
/ocabot merge nobump |
|
What a great day to merge this nice PR. Let's do it! |
|
Congratulations, your PR was merged at 59ffb6b. Thanks a lot for contributing to OCA. ❤️ |
depende de:
é construído a partir do PR da migração do l10n_br_account para a v15: #2806
No momento falta migrar o método account.move#_compute_amount e os dados dos testes foram adaptados para passar mesmo com esse método para migrar. Ou seja: os lançamentos nos account.move.line parecem OK, mas os totais ao nível do account.move ainda precisam de retrabalho.
Tem alguns outros ajustes menores para fazer também.