Skip to content

[16.0][FIX] shopfloor: fix test broken after dependent module update#1149

Merged
OCA-git-bot merged 1 commit intoOCA:16.0from
acsone:16.0-fix-shopfoor-delivery-carrier-update
Mar 30, 2026
Merged

[16.0][FIX] shopfloor: fix test broken after dependent module update#1149
OCA-git-bot merged 1 commit intoOCA:16.0from
acsone:16.0-fix-shopfoor-delivery-carrier-update

Conversation

@nicolas-delbovier-acsone
Copy link
Copy Markdown

@nicolas-delbovier-acsone nicolas-delbovier-acsone commented Mar 30, 2026

Tests recently broke because of this commit in a dependent module: OCA/delivery-carrier@9c8bd5c

See this PR: OCA/delivery-carrier#1142

An outgoing picking is now considered its own "ship_picking_id" (which was not the case before) leading to more data than previously expected in the tests.

@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

@lmignon
Copy link
Copy Markdown

lmignon commented Mar 30, 2026

Tests broke because of this commit in a dependent module: OCA/delivery-carrier@9c8bd5c

See this PR: OCA/delivery-carrier#1142

An outgoing picking is now considered its own "ship_picking_id" (which was not the case before) leading to more data than previously expected in the tests.
@nicolas-delbovier-acsone nicolas-delbovier-acsone force-pushed the 16.0-fix-shopfoor-delivery-carrier-update branch from da438b0 to 3356b6a Compare March 30, 2026 11:34
@nicolas-delbovier-acsone
Copy link
Copy Markdown
Author

@lmignon This is fixed now

@nicolas-delbovier-acsone
Copy link
Copy Markdown
Author

cc @jbaudoux

@jbaudoux
Copy link
Copy Markdown
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-1149-by-jbaudoux-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 797c81f into OCA:16.0 Mar 30, 2026
6 of 7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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