-
-
Notifications
You must be signed in to change notification settings - Fork 719
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
[ADD] stock_picking_batch_packaging_qty: Packaging in stock picking batch #2001
[ADD] stock_picking_batch_packaging_qty: Packaging in stock picking batch #2001
Conversation
6853303
to
4c10a97
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.
Tested in http://oca-stock-logistics-warehouse-16-0-pr2001-4c10a9778559.runboat.odoo-community.org/web
"Base database", just installed and configured the needed.
👍🏼
Please @EmilioPascual review my review in loom! 😄
https://www.loom.com/share/64dc6987c2074f15825bf1701e1d3687?sid=0df658af-aa8f-4e35-a885-25a5d6f11db3
Please @yajo review, you wrote ADP MT-5531. 😄 Looks promising! |
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
4c10a97
to
1d8d0d8
Compare
1d8d0d8
to
ef1fd66
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.
LGTM
/ocabot merge patch |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 37cd39d. Thanks a lot for contributing to OCA. ❤️ |
Added packaging, packaging qty and packaging qty done to views inside batch (tabs operations and detailed operations)
@yajo @Shide @rafaelbn @Gelojr please review. Thank you.
@moduon MT-5810