-
-
Notifications
You must be signed in to change notification settings - Fork 252
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
[16.0][DEL][l10n_br_fiscal]: remove empty mixin lines onchanges #3655
[16.0][DEL][l10n_br_fiscal]: remove empty mixin lines onchanges #3655
Conversation
Hi @renatonlima, |
5da7aa8
to
1f4eb5c
Compare
78a9c57
to
0a8d638
Compare
@renatonlima ok com esse? |
This PR has the |
Eu havia implementado esses métodos para futuramente implementar a funcionalidade de informar os valores dos impostos manualmente, mas como na versão 16.0 é possível fazer com compute, acho que faz sentido remove-los. |
/ocabot merge minor |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at eee7f94. Thanks a lot for contributing to OCA. ❤️ |
Uns anos atrás, o @renatonlima tinha preparado esses onchange nas linhas dos documentos fiscais como pontos de extensões a serem customizados. Porem, eu acho melhor tirar pois:
Eu diria que a única duvida que cabe é se vale a pena deixar esses métodos comentados como eu fiz para documentar que é possível implementar. Não sei se vale bem a pena...