Skip to content

Conversation

@chienandalu
Copy link
Member

@rousseldenis
Copy link
Contributor

/ocabot migration sale_order_product_recommendation_packaging_default

yajo and others added 12 commits January 8, 2026 16:46
This module extends the functionality of sale order product recommendations to
support gathering and specifying product packaging and to allow you to input
that info directly from the recommendations wizard.

To know how to use this module, follow the usage instructions of `sale_order_product_recommendation`.

If your database uses product packagings, you should be able to see and use 2 new columns in the recommendations wizard:

- *Packaging*
- *Packaging Qty.*

@moduon MT-3489
…oduct without packaging

Before this patch, when a product had no packaging, we got an error.

@moduon MT-4326
It turns out that, since product packagings can be sorted, and can already be set as `sales=True`, we can safely consider that the 1st packaging with that field on is the default one for sales.

We loose the ability to set one product with sales packagings to not have a default one, but we win in the sense that we don't diverge from Odoo CE DB model.

That advantage means that the "packaging_default" family of modules are now just a plug-in that will emphasize the usage of packagings in the sale process, but other modules that use the same data model can reason similarly without depending directly on these ones.

@moduon MT-4396
…ckaging fields in mobile views

In this case, the qty is always visible, to avoid UI jumps if the when the user uses buttons from the numeric step widget, if OCA/web#2691 is included.

Also, that module is added to dependencies.

@moduon MT-4397 MT-4396
…te packaging

Now it includes the default product in the context, so when you use quick creation actions, it will behave as expected.

@moduon MT-4472
…date of preexisting lines, and don't fail if there's no packaging permissions

@moduon MT-4472
@chienandalu chienandalu force-pushed the 18.0-mig-sale_order_product_recommendation_packaging_default branch from e6d923f to 0d91075 Compare January 8, 2026 15:46
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.

8 participants