wms icon indicating copy to clipboard operation
wms copied to clipboard

[FIX] stock_available_to_promise_release: Fix priority

Open lmignon opened this issue 1 year ago • 5 comments

When we compute the priority to apply at release time to the released picking, we must ignore priorities from already processed pickings. This is required to avoid corner cases where you explicitely set a new lower priority on a picking to release and this priority is changed since some of your moves are linked to backoder pickings already processed with an higher priority

cc @jbaudoux @rousseldenis @sbejaoui

lmignon avatar Jan 31 '24 16:01 lmignon

/ocabot merge patch

jbaudoux avatar Apr 03 '24 21:04 jbaudoux

On my way to merge this fine PR! Prepared branch 16.0-ocabot-merge-pr-832-by-jbaudoux-bump-patch, awaiting test results.

OCA-git-bot avatar Apr 03 '24 21:04 OCA-git-bot

@jbaudoux your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-832-by-jbaudoux-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

OCA-git-bot avatar Apr 03 '24 21:04 OCA-git-bot

@lmignon Can you rebase ?

jbaudoux avatar Apr 04 '24 07:04 jbaudoux

@lmignon Something to fix here ?

rousseldenis avatar Apr 09 '24 14:04 rousseldenis

/ocabot merge patch

rousseldenis avatar Jun 19 '24 14:06 rousseldenis

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

OCA-git-bot avatar Jun 19 '24 14:06 OCA-git-bot

Congratulations, your PR was merged at 54e1fb0236e4579650e4c63fcfc1974fda7b1f98. Thanks a lot for contributing to OCA. ❤️

OCA-git-bot avatar Jun 19 '24 15:06 OCA-git-bot