Skip to content

[MIG] sale_mrp_link merged into sale_mrp.#5562

Open
legalsylvain wants to merge 1 commit intoOCA:14.0from
grap:14.0-sale_mrp_link-merged-into-sale
Open

[MIG] sale_mrp_link merged into sale_mrp.#5562
legalsylvain wants to merge 1 commit intoOCA:14.0from
grap:14.0-sale_mrp_link-merged-into-sale

Conversation

@legalsylvain
Copy link
Contributor

@legalsylvain legalsylvain added this to the 14.0 milestone Mar 19, 2026
@OCA-git-bot
Copy link
Contributor

Hi @pedrobaeza, @StefanRijnhart, @MiquelRForgeFlow,
some modules you are maintaining are being modified, check this out!

@MiquelRForgeFlow
Copy link
Contributor

Referenced commit adds sale_order_count to mrp.production, but sale_mrp_link adds sale_order_id to mrp.production. I think it's not the same.

@legalsylvain
Copy link
Contributor Author

legalsylvain commented Mar 19, 2026

Referenced commit adds sale_order_count to mrp.production, but sale_mrp_link adds sale_order_id to mrp.production. I think it's not the same.

I'm not expert in mrp features. What I understand :

  • The "feature" has been merged. because the objective of the sale_mrp_link is to provide form links between sale order and mrp production.
  • the implementation is not the same, because before, the link between sale order and mrp production was M2O, and in odoo 14.0 it's a M2M. So indeed, the computation is different, and the sale_order_id is now obsolete, I think.

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.

3 participants