Skip to content

Conversation

@LoisRForgeFlow
Copy link
Contributor

repair_id is reserved to the stock.moves directly related to the execution of the repair order and has associated logic in odoo core code (like assigning the picking type) To avoid unexpected behaviors, we use related_repair_id to signal auxiliar stock moves that are not directly related to repair execution.

To not break existing modules depending on it, support for repair_id is kept but it is recommended to remove in v18.

`repair_id` is reserved to the stock.moves directly related
to the execution of the repair order and has associated logic
in odoo core code (like assigning the picking type)
To avoid unexpected behaviors, we use `related_repair_id` to
signal auxiliar stock moves that are not directly related to
repair execution.

To not break existing modules depending on it, support for
`repair_id` is kept but it is recommended to remove in v18.
@LoisRForgeFlow
Copy link
Contributor Author

cc @ppyczko @DavidJForgeFlow What do you think?

Copy link
Contributor

@DavidJForgeFlow DavidJForgeFlow left a comment

Choose a reason for hiding this comment

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

Makes sense!

Copy link
Contributor

@ppyczko ppyczko left a comment

Choose a reason for hiding this comment

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

LGTM!

@LoisRForgeFlow
Copy link
Contributor Author

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-90-by-LoisRForgeFlow-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 647c767 into OCA:17.0 Feb 17, 2025
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 3bed360. 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.

4 participants