Skip to content

Conversation

@Mohajiro
Copy link
Contributor

@Mohajiro Mohajiro commented Dec 5, 2025

Introduce a technical bridge module ensuring that service lines from all repairs in a group are properly added to sale orders created by grouped repair logic. This restores consistency between repair_order_group and repair_service workflows.

Task: 5107

@Mohajiro Mohajiro force-pushed the 18.0-t5107-repair_order_group_services-new_module branch 2 times, most recently from 9bb7747 to 00536fb Compare December 5, 2025 16:40
Copy link

@Aldeigja Aldeigja left a comment

Choose a reason for hiding this comment

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

Functional LGTM

@@ -0,0 +1 @@
odoo-addon-repair_order_group @ git+https://github.com/OCA/repair.git@refs/pull/136/head#subdirectory=repair_order_group
Copy link
Member

Choose a reason for hiding this comment

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

This module was merged, please remove this file and rebase the branch.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you, it's done.

Introduce a technical bridge module ensuring that service lines from all
repairs in a group are properly added to sale orders created by grouped
repair logic. This restores consistency between repair_order_group and
repair_service workflows.

Task: 5107
@Mohajiro Mohajiro force-pushed the 18.0-t5107-repair_order_group_services-new_module branch from 00536fb to af2c95c Compare December 17, 2025 15:58
Copy link
Member

@ivs-cetmix ivs-cetmix 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 LGTM

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Copy link

@WesleyOliveira98 WesleyOliveira98 left a comment

Choose a reason for hiding this comment

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

LGTM

@ivs-cetmix
Copy link
Member

Dear @OCA/manufacturing-maintainers would be really great to have this bridge module merged!)

Copy link
Contributor

@ivantodorovich ivantodorovich left a comment

Choose a reason for hiding this comment

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

LG. thanks

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-138-by-ivantodorovich-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 50b59a9 into OCA:18.0 Dec 22, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

6 participants