[14.0][IMP] l10n_br_mdfe: add transmission, cancelling and closure

[marcelsavegnago]

No description provided.

[OCA-git-bot]

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

[mileo]

Como o MDF-e é um documento interno da empresa e não representa uma cobrança nem um recebimento nunca. Ele precisa ter vinculo com um account.move?

Não seria melhor ele ser somente um documento fiscal?

[marcelsavegnago]

Como o MDF-e é um documento interno da empresa e não representa uma cobrança nem um recebimento nunca. Ele precisa ter vinculo com um account.move?

Não seria melhor ele ser somente um documento fiscal?

Acredito que hoje quando o l10n_br_account esta instalado, ao criar o documento fiscal o account move ẽ criado junto.

[rvalyi]

Como o MDF-e é um documento interno da empresa e não representa uma cobrança nem um recebimento nunca. Ele precisa ter vinculo com um account.move?

Não seria melhor ele ser somente um documento fiscal?

Acredito que hoje quando o l10n_br_account esta instalado, ao criar o documento fiscal o account move ẽ criado junto.

Nope, cria apenas o documento fiscal no caso. Eh apenas o contrário que acontece se criar um account.move com um fiscal_operation_id ai cria documento fiscal junto.

no final do account_move.py tem uma função para importar account.move a partir de um documento fiscal, mas é um API que tem que ser chamado propositalmente (por examplo na importação de uma NFe).

do restante pode ser que o @mileo tenha razão e que nao devemos criar account.move para a MDFe...

[antoniospneto]

Sobre a questão de atrelar a fatura ao MDFe, se não há nenhuma complicação técnica, concordo que é melhor não criar a fatura.

[marcelsavegnago]

Sobre a questão de atrelar a fatura ao MDFe, se não há nenhuma complicação técnica, concordo que é melhor não criar a fatura.

Boa., eu tbm acho que eh bem melhor mesmo.. Enfim, vamos tratar isso em outra PR e deixar esta aqui em paralelo ja que ela soh trata a transmissao mesmo..ai depois faco rebase

[rvalyi]

parem ele! chega de PR de novo documento fiscal tudo verdinho! parece até que vc gosta desses documentos fiscais.

[marcelsavegnago]

agora sim :D tudo verdim

[marcelsavegnago]

Pronto.. soh mais um push para acertar o commit :D

[marcelsavegnago]

@mileo @antoniospneto @rvalyi sobre a criacao de documento fiscal sem o account.move, fiz alguns testes e de fato da para fazer... hoje do jeito que esta ja eh "possivel".. Porem, precisamos de alguns ajustes para evitar entrada incorreta dependendo do local que o usuario acessar. Sendo assim, nesta PR vou abstrair disso ja que o esforco maior sera mesmo em trabalhar no modulo l10n_br_account e no l10n_br_fiscal para viabilizar. Fiz uma PR de esboco #3536

[rvalyi]

parabens pelo trabalho @marcelsavegnago !

No geral não me parece nada ruim, pelo menos na questão do código. A única coisa que eu peço é da gente ver com calma o merge do l10n_br_mdfe/models/documents.py e as comunicação SOAP que depende do erpbrasil.edoc.

Pois estamos procurando encaixar uma transmissão alternativa pelo cliente SOAP do xsdata e usando os bindings SOAP em vez de manter tudo isso manualmente e ter que fazer mil gambiarras para testar ainda (alias essa parte não tem teste no seu PR, mas talvez seja melhor assim por enquanto).

O @renatonlima tb ta vendo para limpar essas coisas...

E eu começei e experimentar na 16.0 com a transmissão da NFE aqui #3671
Ta bem experimental ainda, mas me permite de mapear um pouco como refatorar as coisas para poder usar o cliente nfelib alternativo (no mesmo molde que fizemos com a impressão pelo erpbrasil.edoc.pdf).

Dificilmente a gente faria um backport disso para a v14 (a não ser que vc quiser fazer), mas seria bom bolar a comunicação SOAP da MDF-e (e da CT-e) que não enche o saco para limpar as coisas na 16.0 depois...

Bem, é apenas para dizer que temos que olhar essa parte com calma. Pode olhar meu PR se quiser começar a pensar sobre. Nessas semanas que vem do feedback sobre isso.

Se quiser separar um pouco essa parte mais controversa da transmissão SOAP, da para fazer o merge do restante antes. Talvez isso da parte do SOAP não seja tão complexo no caso da MDFe, mas fico com pé atras porque no caso da NFe ta dando trabalho encaixar o novo cliente...

cc @antoniospneto @mileo.

[rvalyi]

@marcelsavegnago eu acabei olhando melhor e eu acho OK fazer o merge disso. Vai ter que retrabalhar na hora de portar para a 16.0, mas contudo não vai ser um trabalho tão grande e parece melhor já prosseguir com o merge.

cc @OCA/local-brazil-maintainers

[marcelsavegnago]

ping @antoniospneto @mileo

[rvalyi]

/ocabot merge minor

[OCA-git-bot]

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

[OCA-git-bot]

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