Skip to content
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

[18.0][MIG] sale_stock_mto_as_mts_orderpoint: Migration to 18.0 #1828

Open
wants to merge 31 commits into
base: 18.0
Choose a base branch
from

Conversation

chaule97
Copy link

  • because stock_orderpoint_manual_procurement is deprecated, so I remove it in depends and write more code about stock_warehouse_orderpoint

grindtildeath and others added 30 commits January 2, 2025 15:48
For user without proper rights the confirmation of a sale.order can lead
to an access error on the orderpoint creation.
The sudo fixes that.
On sale confirmation generate the procurement if the rule is on the sale
order line OR the rule is activated on the product.
When generation the orderpoint from the sale order line, it might happen
that the sale_order_line's product is a BOM, the moves that are generated
are based on the components that composes it.

For this reason, when creating the purchase order, we have to get the
product from moves, not sale order lines.
Add a flag `mto_as_mts`on the warehouse to activate the functionality of the module.

It will fix the interferences with other modules in the repository.
And allows for better customization

This a backport from v16 improvements.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: stock-logistics-workflow-14.0/stock-logistics-workflow-14.0-sale_stock_mto_as_mts_orderpoint
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-workflow-14-0/stock-logistics-workflow-14-0-sale_stock_mto_as_mts_orderpoint/
@rousseldenis
Copy link
Contributor

@chaule97 Thanks for this. Orderpoints related module have migrated to : https://github.com/OCA/stock-logistics-orderpoint

Thanks to do the PR there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.