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
235 stars 244 forks source link

[14.0][FIX][l10n_br_nfe] falha no cancelamento de nota fiscal sem retorno #3182

Closed DiegoParadeda closed 1 week ago

DiegoParadeda commented 1 month ago

Acredito que o commit https://github.com/OCA/l10n-brazil/commit/f6d0e9e9cdf061bb03d2b54be4e3f382de59751c 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:

image

OCA-git-bot commented 1 month ago

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

antoniospneto commented 1 week ago

@DiegoParadeda agora está pronto para revisão?

DiegoParadeda commented 1 week ago

@DiegoParadeda agora está pronto para revisão?

Sim @antoniospneto! Tinha esquecido de atualizar aqui

rvalyi commented 1 week ago

/ocabot merge patch

OCA-git-bot commented 1 week ago

This PR looks fantastic, let's merge it! Prepared branch 14.0-ocabot-merge-pr-3182-by-rvalyi-bump-patch, awaiting test results.

OCA-git-bot commented 1 week ago

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

marcelsavegnago commented 5 days ago

@DiegoParadeda pode portar para as versões superiores ?

rvalyi commented 3 days ago

@DiegoParadeda pode portar para as versões superiores ?

feito em #3323 e #3324