-
-
Notifications
You must be signed in to change notification settings - Fork 243
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[14.0][FIX][l10n_br_nfe] falha no cancelamento de nota fiscal sem retorno #3182
Conversation
Hi @renatonlima, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Será que essa validação não é melhor ser diretamente no módulo da NF-e? considerando que para outros documentos fiscais a regra pode ser outra? por exemplo na nota fiscal de serviço do provedor Betha, não há minimo,
830b98e
to
b322417
Compare
@DiegoParadeda agora está pronto para revisão? |
Sim @antoniospneto! Tinha esquecido de atualizar aqui |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/ocabot merge patch |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at b8e72bd. Thanks a lot for contributing to OCA. ❤️ |
@DiegoParadeda pode portar para as versões superiores ? |
|
Acredito que o commit f6d0e9e tenha trazido resultados não desejados uma vez que o campo justificativa (no cancelamento da nota fiscal) precisa ter no mínimo 15 caracteres.
Essa propriedade passava despercebida e, com o commit, o erro passou a ocorrer sem nenhum tipo de retorno para o usuário.
Esse PR trava o cancelamento da nota fiscal caso o tamanho mínimo da justificativa não tenha sido atendida: