-
-
Notifications
You must be signed in to change notification settings - Fork 620
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
[FIX] stock_picking_batch_extended: float_is_zero required parameters #595
[FIX] stock_picking_batch_extended: float_is_zero required parameters #595
Conversation
Hi @Tecnativa, @camptocamp, |
can you update other calls float_is_zero in this addon |
f47d0fb
to
f310380
Compare
return float_is_zero( | ||
operation.product_qty, | ||
precision_rounding=operation.product_uom_id.rounding | ||
) and operation.qty_done or operation.product_qty |
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.
Should not be qty_done tested with float_is_zero as well?
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.
Commented offline :)
@i-vyshnevska is it ok for you? |
/ocabot merge patch |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at c40f5cf. Thanks a lot for contributing to OCA. ❤️ |
@Tecnativa TT18083