-
-
Notifications
You must be signed in to change notification settings - Fork 246
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_fiscal: ncm starting with 09 conflicts with service type #3469
[14.0][FIX] l10n_br_fiscal: ncm starting with 09 conflicts with service type #3469
Conversation
Hi @renatonlima, |
@crsilveira se puder revisar. |
Assim funciona. |
@renatonlima de acordo dessa forma? |
Já tem um tempo que esse onchange da problema, em alguns clientes eu já tinha inclusive desativado. Desta forma ficou melhor. |
cb52f1b
to
4bdb6e9
Compare
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.
Agora tá perfeito, obrigado @CristianoMafraJunior
/ocabot merge patch |
On my way to merge this fine PR! |
@OCA/local-brazil-maintainers Pessoal, eu dei merge pois as alterações já estão conforme @renatonlima sugeriu em #3096 (review) |
Congratulations, your PR was merged at b9b7997. Thanks a lot for contributing to OCA. ❤️ |
Arrumei o codigo conforme foi feita a revisão na PR anterior, se estiver tudo certo e for aprovado faço os fwp.
Substituindo a PR #3096 para fechar a Issue #3095
VIDEO_TESTE_NCM.mp4