-
-
Notifications
You must be signed in to change notification settings - Fork 801
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][MIG] purchase_order_product_recommendation: Migration to version 16.0 #1861
[16.0][MIG] purchase_order_product_recommendation: Migration to version 16.0 #1861
Conversation
- Add flag to be able to load all the products with supplier infos for ther current customer
- If you have multiple warehouses, you can also constrain the recommendations to the deliveries of specific ones.
…in supplierinfo Must be commercial partner, not possible contacts put in the purchase order.
- Give the proper context when the wh filter is active to get the right quantities available for those warehouses.
- When the wizard had a lot of lines, the price_unit computation had a very poor performance whenenever we changed the qty to order (maybe due to some ORM issue). Changing to an initial computation and then to a computation on the onchange method, gets rid of the problem.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: purchase-workflow-12.0/purchase-workflow-12.0-purchase_order_product_recommendation Translate-URL: https://translation.odoo-community.org/projects/purchase-workflow-12-0/purchase-workflow-12-0-purchase_order_product_recommendation/
…default code to avoid crash tests with product_code_unique module installed
- Extract domain build to an independent method for easy extension by third modules - Extract product filter to an independent method for easy extension. - New product internal category filter. - New "Show all products" filter that allows to easily show all purchable products. - Make tests extensible by other modules
… or not purchasable products It includes a tests that exercises the patch.
…y inheritance for seller values.
After commit odoo/odoo@4625f3d the warehouse is taken into account when computing the quantities. As we are limiting the test by warehouse, the final value should be updated to the new correctly computed.
…duct code Added product name and product code on wizard lines for allow ordering by both regardless of how it is displayed in display_name
7085492
to
478bc0b
Compare
ping @sergio-teruel and @pedrobaeza |
/ocabot migration purchase_order_product_recommendation |
purchase_order_product_recommendation/wizards/purchase_order_recommendation_view.xml
Outdated
Show resolved
Hide resolved
478bc0b
to
8933992
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.
Change the width to fill all the space.
purchase_order_product_recommendation/wizards/purchase_order_recommendation_view.xml
Outdated
Show resolved
Hide resolved
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 :)
This PR has the |
8933992
to
7424cca
Compare
/ocabot merge nobump |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 9a43e61. Thanks a lot for contributing to OCA. ❤️ |
ping @Juangomezzurita |
@Tecnativa TT42423