[14.0][IMP] l10n_br_fiscal, l10n_br_nfe: atualização do parâmetro do tipo de emissão na geração da chave edoc

[ygcarvalh]

Fiz isso momentaneamente para conseguir fazer o processo da emissão da NFC-e em contingência sem nenhum problema, considerando que há outras formas de tipos de emissão no documento fiscal. Entretanto, eu acabei inserindo esse campo que é adicionado através do módulo l10n_br_nfe para o preenchimento.

A ideia é que nos próximos dias eu implemente esse campo e remova esse TODO presente no código, deixando o código o mais correto possível.

[OCA-git-bot]

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

[ygcarvalh]

Bom, acabei fazendo o processo de alteração do campo entre os módulos, removendo do módulo original e alterando o nome pra ficar padronizado com os demais. Além disso, pelos testes que foram realizados, não houve nenhum problema no fluxo.

A explicação para isso foi a mesma do começo do PR: esse tipo de emissão tem influência na maneira como a chave edoc é gerada, e por isso não pode ser um valor fixado, ao mesmo tempo que não podemos depender do módulo da NFe para conseguir realizar essa configuração. Além disso, no próprio document.py do l10n_br_pos temos um campo related com esse valor e que vai para o edoc.

Críticas e sugestões são sempre bem-vindas.

@mileo @rvalyi @renatonlima @marcelsavegnago

[mileo]

Eu colocaria o prefixo edoc no nome do campo para condizer com os demais.

Além disso falta o migration pois o campo foi renomeado.

[renatonlima]

Pessoal,

Não parece correta essa alteração, pois esse campo não pode ser genérico, deveria ser por tipo de documento fiscal, mesmo porque se uma empresa que emite NF-e e CT-e por exemplo, ele pode emitir NF-e em contingência e emitir os CT-es normalmente.

[ygcarvalh]

Entendo sua preocupação sobre, mas também não podemos deixar o campo fixado para ser sempre emissão normal durante a geração da chave edoc, pois isso atrapalha em casos de contingência. Você teria alguma sugestão de como essa alteração poderia ser feita?

@renatonlima

[marcelsavegnago]

@mileo pode verificar o pre-commit e os testes por favor ?

[antoniospneto]

@ygcarvalh @mileo se puder por a pr em rascunho até ficar pronta para revisão

[mileo]

Já tinha coisa pra arrumar aqui, e com o merge da refatoração no PR #3081 tem mais coisa ainda, sobrando um tempo voltamos nisso.

[github-actions]

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.