Skip to content

[ADD] repair_stock_consumption_step#123

Open
sbejaoui wants to merge 12 commits intoOCA:16.0from
acsone:16.0-repair_stock_consumption_step-sbj
Open

[ADD] repair_stock_consumption_step#123
sbejaoui wants to merge 12 commits intoOCA:16.0from
acsone:16.0-repair_stock_consumption_step-sbj

Conversation

@sbejaoui
Copy link
Copy Markdown

In the base repair module, consumption moves for spare parts are created and immediately validated when the repair order is marked as done. This prevents user interaction such as choosing lot/serial numbers.

This module introduces an optional intermediate step:

  • When enabled at warehouse level, repair consumption moves are grouped into a stock picking.
  • The repair order is set to a new Consumption state until the picking is validated.
  • Users can process the picking manually, assign lots/serials, and only then complete the repair.

cc/ @lmignon , @rousseldenis

@rousseldenis rousseldenis added this to the 16.0 milestone Aug 29, 2025
@sbejaoui sbejaoui force-pushed the 16.0-repair_stock_consumption_step-sbj branch from ddc5d42 to 0527711 Compare August 29, 2025 09:50
Copy link
Copy Markdown

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

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

LGTM

@sbejaoui sbejaoui force-pushed the 16.0-repair_stock_consumption_step-sbj branch from 0527711 to aa305f9 Compare September 1, 2025 08:29
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 4, 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 Jan 4, 2026
Copy link
Copy Markdown

@lmignon lmignon left a comment

Choose a reason for hiding this comment

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

LGTM (Code review + functional)

@github-actions github-actions Bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jan 11, 2026
@sbejaoui sbejaoui force-pushed the 16.0-repair_stock_consumption_step-sbj branch 4 times, most recently from 833666c to 8f1bb67 Compare February 26, 2026 09:56
@nicolas-delbovier-acsone nicolas-delbovier-acsone force-pushed the 16.0-repair_stock_consumption_step-sbj branch from 6d52a43 to 019c179 Compare April 10, 2026 12:24
@OCA-git-bot OCA-git-bot added mod:repair_stock_consumption_step Module repair_stock_consumption_step series:16.0 labels Apr 14, 2026
sbejaoui and others added 10 commits April 15, 2026 08:01
In the base repair module, consumption moves for spare parts are
created and immediately validated when the repair order is marked as done.
This prevents user interaction such as choosing lot/serial numbers.

This module introduces an optional intermediate step:

- When enabled at warehouse level, repair consumption moves are grouped
  into a stock picking.
- The repair order is set to a new **Consumption** state until the picking
  is validated.
- Users can process the picking manually, assign lots/serials, and only
  then complete the repair.
@nicolas-delbovier-acsone nicolas-delbovier-acsone force-pushed the 16.0-repair_stock_consumption_step-sbj branch from fd4dbe9 to e999245 Compare April 15, 2026 06:02
Copy link
Copy Markdown

@lmignon lmignon left a comment

Choose a reason for hiding this comment

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

@nicolas-delbovier-acsone I found a little bug into your last changes...

Comment on lines +34 to +35
for pick in self:
repair = pick.consumption_repair_order_id
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@nicolas-delbovier-acsone Be careful to only take into account piking linked to a repair order.

if partial_consumption_picks := partially_processed_pickings.filtered(
"consumption_repair_order_id"
):
self._check_no_partial_qties_on_repaired_product()
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

partial_consumption_picks._check_no_partial_qties_on_repaired_product()

@nicolas-delbovier-acsone nicolas-delbovier-acsone force-pushed the 16.0-repair_stock_consumption_step-sbj branch from e999245 to 1659123 Compare April 15, 2026 06:44
@nicolas-delbovier-acsone nicolas-delbovier-acsone force-pushed the 16.0-repair_stock_consumption_step-sbj branch from 1659123 to 350995f Compare April 15, 2026 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants