Skip to content

[16.0][BPORT] shopfloor(_reception): drop dangerous action assign - #1257

Merged
OCA-git-bot merged 3 commits into
OCA:16.0from
jbaudoux:16-bport-fix-shopfloor-action_assign
Sep 15, 2026
Merged

OCA-git-bot merged 3 commits into
OCA:16.0from
jbaudoux:16-bport-fix-shopfloor-action_assign

Conversation

@jbaudoux

@jbaudoux jbaudoux commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @mmequignon, @JuMiSanAr, @simahawk, @guewen, @sebalix,
some modules you are maintaining are being modified, check this out!

@OCA-git-bot OCA-git-bot added series:16.0 mod:shopfloor Module shopfloor mod:shopfloor_reception Module shopfloor_reception labels Sep 7, 2026
@jbaudoux jbaudoux changed the title [16.0][BPORT] shopfloor(_reception): drop dangereous action assign [16.0][BPORT] shopfloor(_reception): drop dangerous action assign Sep 7, 2026
Comment thread shopfloor_reception/tests/test_set_destination.py Outdated
@jbaudoux
jbaudoux force-pushed the 16-bport-fix-shopfloor-action_assign branch 2 times, most recently from d173aab to 802e7b6 Compare September 7, 2026 10:54

@lmignon lmignon left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thank you !

Comment thread shopfloor_reception/services/reception.py
@OCA-git-bot

Copy link
Copy Markdown
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). 🤖

When a confirmed move is split, do not call _action_confirm, not _action_assign on the new move
When a confirmed move is split, do not call _action_confirm, nor _action_assign on the new move
@jbaudoux
jbaudoux force-pushed the 16-bport-fix-shopfloor-action_assign branch from 802e7b6 to ecfea30 Compare September 15, 2026 13:28
@jbaudoux

Copy link
Copy Markdown
Contributor Author

rebased

@lmignon

lmignon commented Sep 15, 2026

Copy link
Copy Markdown

✌️

@sebalix

sebalix commented Sep 15, 2026

Copy link
Copy Markdown

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-1257-by-sebalix-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 56690bd into OCA:16.0 Sep 15, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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