Skip to content

[18.0][MIG] sale_stock_picking_variable_qty: Migration to 18.0 - #2301

Closed
chienandalu wants to merge 3 commits into
OCA:18.0from
moduon:18.0-mig-sale_stock_picking_variable_qty
Closed

[18.0][MIG] sale_stock_picking_variable_qty: Migration to 18.0 #2301
chienandalu wants to merge 3 commits into
OCA:18.0from
moduon:18.0-mig-sale_stock_picking_variable_qty

Conversation

@chienandalu

@chienandalu chienandalu commented Apr 7, 2026

Copy link
Copy Markdown
Member

cc @moduon MT-12702

BhaveshHeliconia
BhaveshHeliconia approved these changes Apr 17, 2026
View reviewed changes

@BhaveshHeliconia BhaveshHeliconia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Functional review LGTM!

@dnplkndll dnplkndll mentioned this pull request Jun 20, 2026
Closed
@rousseldenis

rousseldenis commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

/ocabot migration sale_stock_picking_variable_qty

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Aug 18, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request Aug 18, 2026
Open
@rousseldenis

rousseldenis commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

/ocabot merge nobump

@OCA-git-bot

OCA-git-bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

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

@OCA-git-bot

OCA-git-bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

@rousseldenis your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-2301-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

OCA-git-bot commented Aug 18, 2026

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). 🤖

@chienandalu

chienandalu commented Aug 18, 2026

Copy link
Copy Markdown
Member Author

Closing in favour of stock_picking_variable_qty that provides an approach that doesn't rely on the sale order

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rousseldenis rousseldenis rousseldenis approved these changes
+1 more reviewer
@BhaveshHeliconia BhaveshHeliconia BhaveshHeliconia approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved ready to merge

Projects

None yet

Milestone

18.0

Development

Successfully merging this pull request may close these issues.

4 participants

@chienandalu @rousseldenis @OCA-git-bot @BhaveshHeliconia