search

OCA / l10n-brazil

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

[14.0][FIX] l10n_br_sale_stock: Caso do Pedido de Venda Sem Operação ou Internacional não deve incluir os Dados Fiscais do Brasil #3325

Closed mbcosta closed 2 months ago

mbcosta commented 2 months ago

Case without Fiscal OP.

Caso do Pedido de Venda Sem Operação ou Internacional não deve incluir os Dados Fiscais do Brasil.

Aqui ainda não está sendo incluído o teste porque ocorre um erro com o método Default do stock.picking como precisei alterar isso vou fazer em outro PR logo em seguida, assim deve facilitar a revisão.

EDIT.: Esse PR está buscando extrair partes das alterações do PR de Extração para o sale_stock_picking_invoicing https://github.com/OCA/l10n-brazil/pull/2955

cc @rvalyi @renatonlima @marcelsavegnago @mileo @antoniospneto

OCA-git-bot commented 2 months ago

Hi @renatonlima, some modules you are maintaining are being modified, check this out!

mbcosta commented 2 months ago

valeu @mileo , ao alterar um Pedido que tem Operação Fiscal para Vazio os campos nas Linhas passam para Invisíveis na Visão, dessa forma o Usuário não tem como apagar, o melhor talvez seja que ao apagar o campo no cabeçalho do objeto também seja apagado na linhas( ou "limpar" todos campos Fiscais?), mas para evitar erros eu estou utilizando o campo que o Usuário tem acesso e define que aquele objeto não deve ter Operação Fiscal.

rvalyi commented 2 months ago

/ocabot merge patch

OCA-git-bot commented 2 months ago

On my way to merge this fine PR! Prepared branch 14.0-ocabot-merge-pr-3325-by-rvalyi-bump-patch, awaiting test results.

OCA-git-bot commented 2 months ago

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