Skip to content

[18.0][IMP] repair_service: sync on existing quotation, similar Odoo does for new components added#127

Open
AaronHForgeFlow wants to merge 1 commit intoOCA:18.0from
ForgeFlow:18.0-imp-repair_service-update
Open

[18.0][IMP] repair_service: sync on existing quotation, similar Odoo does for new components added#127
AaronHForgeFlow wants to merge 1 commit intoOCA:18.0from
ForgeFlow:18.0-imp-repair_service-update

Conversation

@AaronHForgeFlow
Copy link
Copy Markdown
Contributor

When the quotation is already created and there is a new service or a change in the quantity we should update the sale order line related to it. Odoo does similar when adding or removing materials.

@AaronHForgeFlow AaronHForgeFlow force-pushed the 18.0-imp-repair_service-update branch 2 times, most recently from d29fa7c to b22e600 Compare October 8, 2025 15:17
Copy link
Copy Markdown
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.

Seems good

@AaronHForgeFlow AaronHForgeFlow force-pushed the 18.0-imp-repair_service-update branch from b22e600 to 039713b Compare October 28, 2025 11:56
@ivs-cetmix
Copy link
Copy Markdown
Member

Hey @AaronHForgeFlow thank you for your contribution! Could you please fix the pre-commit and also improve the test coverage if possible?

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 5, 2026

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions Bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Apr 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs review stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants