-
-
Notifications
You must be signed in to change notification settings - Fork 718
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
[15.0][MIG] stock_inventory_lockdown #2155
[15.0][MIG] stock_inventory_lockdown #2155
Conversation
This module is a feature extracted from stock_inventory_location. Ported to v8 and new API Adopted latest OCA standards Added tests
in stock move instead of checks in quants. This will allow for a more robust lockdown, and still make it possible to perform inventory adjustments in the locked location. Also resolves an outstanding issue related to the previous design not allowing inventory adjustments where negative quants existed.
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
The virtual adjustment location may be different from the one provided in property_stock_inventory of the product.
Currently translated at 100.0% (5 of 5 strings) Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_inventory_lockdown Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_inventory_lockdown/zh_CN/
Currently translated at 100.0% (5 of 5 strings) Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_inventory_lockdown Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_inventory_lockdown/pt_BR/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: stock-logistics-warehouse-13.0/stock-logistics-warehouse-13.0-stock_inventory_lockdown Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-13-0/stock-logistics-warehouse-13-0-stock_inventory_lockdown/
The default order of stock.location is 'complete_name'. Changing it to "name" can have side effects, and shouldn't be required for stock_inventory_lockdown. See https://github.com/OCA/stock-logistics-warehouse/pull/811/files#r512797834
f87db3b
to
16a9ff3
Compare
…n stock.move.line confirmation
…tion if child bin location is being reviwed
16a9ff3
to
cc9166e
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.
Functional review LGTM
I think it would be good to add a test case for the latest changes.
/ocabot migration stock_inventory_lockdown |
/ocabot merge nobump |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at 837bcd9. Thanks a lot for contributing to OCA. ❤️ |
Backport of this migration: #1849
After merging this PR we will reopen the one in 16.0