[14.0][IMP] l10n_br_fiscal,nfe: gCred tag for cBenef (Crédito Presumido)

[corredato]

Esse PR insere as tags gCred, cCredPresumido, pCredPresumido e vCredPresumido quando há um produto cujo tipo de benefício fiscal é credito presumido.

[mbcosta]

@corredato existe um problema no seu PR que não é relacionado ao código em si ou o que você está buscando resolver que é criar um commit único alterando mais de um modulo, isso é evitado para permitir cherry-pick Back Foward Ports de forma simples e "limpa" em cada módulo, por isso a alteração precisa ser dividida em dois commits um no l10n_br_fiscal e outro no l10n_br_nfe, eu não avaliei suas alterações mas essa separação provavelmente vai ser pedida pelos mantenedores dos módulos

[OCA-git-bot]

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

[corredato]

@corredato existe um problema no seu PR que não é relacionado ao código em si ou o que você está buscando resolver que é criar um commit único alterando mais de um modulo, isso é evitado para permitir cherry-pick Back Foward Ports de forma simples e "limpa" em cada módulo, por isso a alteração precisa ser dividida em dois commits um no l10n_br_fiscal e outro no l10n_br_nfe, eu não avaliei suas alterações mas essa separação provavelmente vai ser pedida pelos mantenedores dos módulos

Resolvido, valeu

[mileo]

O PR aparentemente resolve o problema, mas se é uma taxa, o correto não é que ele seja realmente uma taxa no sistema como os outros impostos?

[corredato]

O PR aparentemente resolve o problema, mas se é uma taxa, o correto não é que ele seja realmente uma taxa no sistema como os outros impostos?

Pode me dar um exemplo para eu entender melhor?

[mileo]

O PR aparentemente resolve o problema, mas se é uma taxa, o correto não é que ele seja realmente uma taxa no sistema como os outros impostos?

Pode me dar um exemplo para eu entender melhor?

O FCP por exemplo é uma taxa que só ocorre em algumas ocasiões. Tem bons exemplos no código de como ela é implementada e é fácil e localizar.

[antoniospneto]

@corredato, a tag “[ADD]” que você está utilizando na mensagem de commit é usada apenas quando se adiciona um módulo novo. No seu caso, como é uma nova funcionalidade, a tag correta é “[IMP]”, do inglês “improvement”, que significa melhoria.

[antoniospneto]

Concordo com o @mileo, o valor do "Crédito Presumido do ICMS" pode ser calculado da mesma forma que já é feito nas outras linhas de impostos, poderia se criar um grupo de imposto chamado "ICMS - Crédito Presumido"

[corredato]

@mileo @antoniospneto Fechou, vou olhar como é feito com o FCP e tomar como exemplo, assim que possível já subo as alterações, valeu galera