Skip to content

[18.0][IMP] sale_purchase_force_vendor: propagate the vendor on stock rules#2575

Open
marielejeune wants to merge 1 commit intoOCA:18.0from
acsone:18.0-imp-sale_purchase_force_vendor-mle
Open

[18.0][IMP] sale_purchase_force_vendor: propagate the vendor on stock rules#2575
marielejeune wants to merge 1 commit intoOCA:18.0from
acsone:18.0-imp-sale_purchase_force_vendor-mle

Conversation

@marielejeune
Copy link

@marielejeune marielejeune commented Feb 18, 2025

Up to now the addon is designed to work only with the default MTO + buy routes.
This PR aims at propagating the forced vendor (on the SOL) through stock rules to be able to use more complex MTO-like routes.

See issue #2574 for more details about the target.

Feel free to review/comment. I'm open to discussions as I'm not a stock expert.

Depends on #2523 and on OCA/sale-workflow#3590

class StockMove(models.Model):
_inherit = "stock.move"

propagated_sale_line_id = fields.Many2one(
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Author

Choose a reason for hiding this comment

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

@marielejeune marielejeune force-pushed the 18.0-imp-sale_purchase_force_vendor-mle branch from fee4373 to 40e0ce5 Compare February 25, 2025 09:23
@rousseldenis
Copy link
Contributor

@marielejeune Could you rebase ?

@rousseldenis
Copy link
Contributor

@marielejeune Could you update as OCA/sale-workflow#3590 is merged ?

@marielejeune marielejeune force-pushed the 18.0-imp-sale_purchase_force_vendor-mle branch from 3a9217b to 9cf0b28 Compare October 13, 2025 12:25
@marielejeune
Copy link
Author

@rousseldenis done

@rousseldenis rousseldenis added this to the 18.0 milestone Oct 13, 2025
Copy link
Member

@HviorForgeFlow HviorForgeFlow 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

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.

3 participants