l10n-brazil
l10n-brazil copied to clipboard
OCA
[14.0][MIG] l10n_br_website_sale_delivery
@mileo alguma ideia de como aumentar a cobertura de teste deste módulo ? consegue dar uma força ?
Coloquei a PR como rascunho porque aparentemente precisa de um trabalho neste método
https://github.com/odoo/odoo/blob/1c938d1ea138c083436f108a25c121a6279ef210/addons/website_sale_delivery/controllers/main.py#L41
@rvalyi @renatonlima @mbcosta estou prototipando algo aqui nesta PR. Vcs visualizam algum problema verificar se o custo de entrega é por linha ou total através do campo do documento ao invés de verificar na empresa ? se for o caso podemos adicionar o campo na aba Outros do pedido de venda para que o usuário possa alterar o parametro direto no documento.
https://github.com/OCA/l10n-brazil/pull/1988/commits/6f93da84f38559cf10aab35c5bf1e55cb9f6fa3a
ola @marcelsavegnago a definição sobe os "Custos de Entrega" ser por empresa acredito que foi feito porque em muitos casos é um padrão, mas entendo a necessidade de alterar em um documento especifico, para atender isso acredito que o melhor seja manter o parametro como prioridade e incluir um segundo campo no Documento para se for selecionado permitir alterar o padrão em um documento especifico
@mbcosta fiz um commit meio tosco para fazer uns testes.. se puder dar uma olhada eu agradeço. Se for o caso vou dividir os commits em outras PRs
https://github.com/OCA/l10n-brazil/pull/1988/commits/a7e52b6459e49409afc2f9936076f1dd0d34c898
Agora acho que ta pronto para revisão :D
Tem alguns commits que posso separar e propor em outras PRs. Mass acho que para teste a forma como está permite testes mais abrangentes.
Vou fazer um rebase.. parece que o runboat agora foi afetado pelo produto4d
@marcelsavegnago a velha historia da instalacao do fiscal na localização: https://runboat.odoo-community.org/api/v1/builds/bd8695ef3-f2f4-4bc9-9383-9c59ebe49136/init-log
Fui rodar a sua PR no runboat e a maquina simplesmente não inicializa, o erro que dá é este aqui
Não consegui testar ela localmente ainda, mas vou demorar um pouco mais.
@marcelsavegnago a velha historia da instalacao do fiscal na localização: https://runboat.odoo-community.org/api/v1/builds/bd8695ef3-f2f4-4bc9-9383-9c59ebe49136/init-log
Fui rodar a sua PR no runboat e a maquina simplesmente não inicializa, o erro que dá é este aqui
Não consegui testar ela localmente ainda, mas vou demorar um pouco mais.
Isto é um problema recente. este produto4d foi removido dos dados demo do Odoo e este produto era utilizado pelo l10n_br_fiscal. O Neto removeu este produto do fiscal e agora com o rebase desta branch acho que jaja vc consegue rodar normalmente.
@marcos-mendez nao sei se vc costuma acessar o runboat com frequencia mas dificilmente vc teria este tipo de erro que aconteceu nesta instancia e portanto não vejo muita veracidade na afirmação sobre "a velha história"
@marcos-mendez nao sei se vc costuma acessar o runboat com frequencia mas dificilmente vc teria este tipo de erro que aconteceu nesta instancia e portanto não vejo muita veracidade na afirmação sobre "a velha história"
Eu reiniciei ela, inicializou de novo com reset e mesma coisa
Velha historia porque é um erro bem comum quando tem dados demo no fiscal e não é a primeira vez que vejo isso, no env local eu já tive isso e já vi isso, por isso disse velha historia. Mandei rebuildar de novo vamos ver o que que acontece.
@marcos-mendez nao sei se vc costuma acessar o runboat com frequencia mas dificilmente vc teria este tipo de erro que aconteceu nesta instancia e portanto não vejo muita veracidade na afirmação sobre "a velha história"
Eu reiniciei ela, inicializou de novo com reset e mesma coisa
Velha historia porque é um erro bem comum quando tem dados demo no fiscal e não é a primeira vez que vejo isso, no env local eu já tive isso e já vi isso, por isso disse velha historia. Mandei rebuildar de novo vamos ver o que que acontece.
eu fiz o rebase do código.. veja que o runboat está no ar agora..
Acho que foi, vou testar agora
o problema iniciou aqui https://github.com/odoo/odoo/pull/101920
o problema iniciou aqui
This is PR is not yet merged, so I doubt it is the cause of your problem.
o problema iniciou aqui
This is PR is not yet merged, so I doubt it is the cause of your problem.
For some reason we started having a problem with the product product.product_product_4d and we had to remove this product from the demo data of the Brazilian localization. Basically we use product.product_product_4c instead of 4d
o problema iniciou aqui
This is PR is not yet merged, so I doubt it is the cause of your problem.
@pivi-odoo This is the commit that got merged into 14.0 https://github.com/odoo/odoo/commit/ab1d7caefa910197579b12ae934fb572f9cff50d cc @marcelsavegnago not a big deal to adapt the localization tests/demo data but definitely the reason why things broke downstream.
Yeah I just saw that it was merged, but you would have had problems nonetheless, since installing Sales deletes the product, and any other demo data that references it would fail to install. This is the reason why the PR was made. Just avoid relying on this particular variant :)
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-1988-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-1988-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-1988-by-marcelsavegnago-bump-nobump, awaiting test results.
@marcelsavegnago your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-1988-by-marcelsavegnago-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-1988-by-marcelsavegnago-bump-nobump, awaiting test results.