wms icon indicating copy to clipboard operation
wms copied to clipboard

[16.0][MIG] stock_picking_batch_creation

Open lmarion-source opened this issue 3 years ago • 7 comments

Depends on:

  • [x] https://github.com/OCA/stock-logistics-warehouse/pull/1626

lmarion-source avatar Jan 13 '23 07:01 lmarion-source

cc @jbaudoux

lmignon avatar Jan 17 '23 15:01 lmignon

@JuMiSanAr @jbaudoux In the las commit I fully refactored the code to improves performances, avoid concurrency issues is the most efficient way and only rely on SQL queries to apply the constraints when looking for picking to add to a batch.

lmignon avatar Feb 04 '23 18:02 lmignon

  • [ ] depends on https://github.com/OCA/stock-logistics-warehouse/pull/1626

lmignon avatar Feb 04 '23 18:02 lmignon

ping @rousseldenis

lmignon avatar Feb 04 '23 18:02 lmignon

@simahawk @jbaudoux The history is cleaned up

lmignon avatar Mar 10 '23 14:03 lmignon

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[bot] avatar Apr 14 '24 14:04 github-actions[bot]

@lmignon @rousseldenis Can you update your review? I see all comments are outdated

jbaudoux avatar Apr 15 '24 07:04 jbaudoux

/ocabot merge nobump

rousseldenis avatar May 02 '24 11:05 rousseldenis

On my way to merge this fine PR! Prepared branch 16.0-ocabot-merge-pr-534-by-rousseldenis-bump-nobump, awaiting test results.

OCA-git-bot avatar May 02 '24 11:05 OCA-git-bot

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

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 May 02 '24 11:05 OCA-git-bot

po file fixed and branch rebased

lmignon avatar May 02 '24 11:05 lmignon

/ocabot merge nobump

rousseldenis avatar May 02 '24 11:05 rousseldenis

On my way to merge this fine PR! Prepared branch 16.0-ocabot-merge-pr-534-by-rousseldenis-bump-nobump, awaiting test results.

OCA-git-bot avatar May 02 '24 11:05 OCA-git-bot

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

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 May 02 '24 12:05 OCA-git-bot

/ocabot merge nobump

rousseldenis avatar May 02 '24 13:05 rousseldenis

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-534-by-rousseldenis-bump-nobump, awaiting test results.

OCA-git-bot avatar May 02 '24 13:05 OCA-git-bot

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

OCA-git-bot avatar May 02 '24 13:05 OCA-git-bot