OCA
[14.0][MIG] l10n_br_sale_stock
Migração do módulo l10n_br_sale_stock depende do PR https://github.com/OCA/l10n-brazil/pull/1937 (assim que esse PR estiver verde eu já coloco os commits nesse PR para continuar a migração)
@renatonlima tem alguns commits interessantes que o @mbcosta fez na PR de migração para 13.0 #https://github.com/OCA/l10n-brazil/pull/1786. Pode verificar e se possivel fazer o cherry-pick para a branch desta PR fazendo favor?
https://github.com/OCA/l10n-brazil/pull/1786/commits/0b0c04503764352db55843665d9f3470a01e378e https://github.com/OCA/l10n-brazil/pull/1786/commits/112d1be0abf037181f67a6663c58f93f111963e8
ola @marcelsavegnago eu fiz os cherry-pick que você mencionou, o PR está verde e acredito que já é possível fazer a revisão, alguns pontos:
-
O campo do Endereço de Entrega/partner_shipping_id já está vindo preenchido pelo modulo stock_picking_invoicing mesmo quando o campo tem o mesmo valor do partner_id, mas esse caso na v12 o campo hoje vai vazio, existe um teste tanto na v12 https://github.com/OCA/l10n-brazil/blob/12.0/l10n_br_sale_stock/tests/test_sale_stock.py#L469 quanto na v14 https://github.com/akretion/l10n-brazil/blob/14.0-mig-l10n_br_sale_stock/l10n_br_sale_stock/tests/test_sale_stock.py#L486 que confirmam isso, afim de evitar que na NFe criada esteja com o campo de Endereço de Entrega preenchido quando é o mesmo do Endereço do Destinatario eu estou apagando essa chave no commit https://github.com/OCA/l10n-brazil/pull/1961/commits/7de8d31b96b4998209acbe26376c20e6a76e2420 coloquei um TODO para confirmação, mas pelo o que entendo é melhor manter o mesmo funcionamento da v12 e ter a NFe sem isso por ser desnecessário nesse caso, mas é um assunto que pode ser debatido
-
Foi incluído no l10n_br_stock_account o Diário para Devoluções de Vendas, devido a questão de apenas os Diários do Tipo sale e purchase poderem gerar as Faturas e nos testes do l10n_br_sale_stock inclui a empresa do Lucro Presumido porque nos testes do travis o caso do main_company falha um Browse nos impostos( mesmo problema comentado no l10n_br_stock_account ), erro que pelo o que parece falha devido a forma de instalação dos módulos mas que não acontece no caso Lucro Presumido, portanto parece ser algo especifico do travis
-
Havia a ideia de extrair boa parte do modulo para o repo account-invoicing https://github.com/OCA/account-invoicing/pull/1025 mas é preciso confirmar se existe o interesse dos mantedores lá nesse modulo e hoje acredito que perdemos a "janela" para fazer essa extração e será melhor ver isso na futura v16
cc @rvalyi @renatonlima @mileo
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). 🤖
What a great day to merge this nice PR. Let's do it! Prepared branch 14.0-ocabot-merge-pr-1961-by-rvalyi-bump-nobump, awaiting test results.
@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-1961-by-rvalyi-bump-nobump.
After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.
What a great day to merge this nice PR. Let's do it! Prepared branch 14.0-ocabot-merge-pr-1961-by-rvalyi-bump-nobump, awaiting test results.
@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-1961-by-rvalyi-bump-nobump.
After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.
On my way to merge this fine PR! Prepared branch 14.0-ocabot-merge-pr-1961-by-rvalyi-bump-nobump, awaiting test results.
@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-1961-by-rvalyi-bump-nobump.
After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.
erro de login estranho na hora de fazer o push dos arquivos .pot :
Entao eu parti para atualizar a CI para ver se resolvia: https://github.com/OCA/l10n-brazil/pull/2129
What a great day to merge this nice PR. Let's do it! Prepared branch 14.0-ocabot-merge-pr-1961-by-rvalyi-bump-nobump, awaiting test results.
Congratulations, your PR was merged at 1659f1469101ffa7044281beb653d002b52e09fc. Thanks a lot for contributing to OCA. ❤️