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
[8.0] remove update stock move line when create invoice #499
[8.0] remove update stock move line when create invoice #499
Conversation
@sergiocorato Could you maybe add a test to enlight the change ? |
4f1c7d2
to
00b33ef
Compare
@sergiocorato Could you update modules version and add some tests ? |
@rousseldenis ok version, as it is an improvement on execution time, add some time-execution-test? |
00b33ef
to
ce8eb17
Compare
[8.0] remove update of stock move when invoicing [IMP] module version
ce8eb17
to
64ae48e
Compare
@sergiocorato Do you plan to add tests or is this ready ? |
@rousseldenis I think we can proceed as-is |
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.
Code review
/ocabot merge patch |
On my way to merge this fine PR! |
Congratulations, your PR was merged at 20989f1. Thanks a lot for contributing to OCA. ❤️ |
WIP test todo