Skip to content

[15.0][MIG] purchase_order_product_recommendation: Migration to 15.0#1518

Merged
OCA-git-bot merged 34 commits intoOCA:15.0from
Tecnativa:15.0-mig-purchase_order_product_recommendation
Jul 13, 2022
Merged

[15.0][MIG] purchase_order_product_recommendation: Migration to 15.0#1518
OCA-git-bot merged 34 commits intoOCA:15.0from
Tecnativa:15.0-mig-purchase_order_product_recommendation

Conversation

@cesar-tecnativa
Copy link
Copy Markdown
Contributor

@Tecnativa TT36506

chienandalu and others added 30 commits July 6, 2022 13:07
- 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
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/
… or not purchasable products

It includes a tests that exercises the patch.
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
@cesar-tecnativa
Copy link
Copy Markdown
Contributor Author

Depends on #1517

@cesar-tecnativa
Copy link
Copy Markdown
Contributor Author

@ernestotejeda @pedrobaeza could you please review?

@pedrobaeza
Copy link
Copy Markdown
Member

/ocabot migration purchase_order_product_recommendation

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Jul 10, 2022
@OCA-git-bot OCA-git-bot mentioned this pull request Jul 10, 2022
78 tasks
Copy link
Copy Markdown
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code review

@cesar-tecnativa
Copy link
Copy Markdown
Contributor Author

@CarlosRoca13 please review

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@pedrobaeza
Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 15.0-ocabot-merge-pr-1518-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Jul 13, 2022
Signed-off-by pedrobaeza
@OCA-git-bot
Copy link
Copy Markdown
Contributor

It looks like something changed on 15.0 in the meantime.
Let me try again (no action is required from you).
Prepared branch 15.0-ocabot-merge-pr-1518-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit d9b6632 into OCA:15.0 Jul 13, 2022
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 1d7dce8. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.