Skip to content

[16.0][MIG] stock_split_picking #1107

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 39 commits into from
Jan 24, 2023
Merged

[16.0][MIG] stock_split_picking #1107

merged 39 commits into from
Jan 24, 2023

Conversation

ps-tubtim
Copy link
Member

Standard Migration.

@oca-clabot
Copy link

Hey @ps-tubtim, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: https://odoo-community.org/page/cla
Here is a list of the users:

Appreciation of efforts,
OCA CLAbot

@ps-tubtim ps-tubtim force-pushed the 16.0-mig-stock_split_picking branch from 5b22593 to 535aa6c Compare October 10, 2022 10:22
@ps-tubtim ps-tubtim mentioned this pull request Oct 10, 2022
Open
75 tasks
@rousseldenis
Copy link
Contributor

/ocabot migration stock_split_picking

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Oct 10, 2022
dannyadair
dannyadair approved these changes Nov 9, 2022
View reviewed changes
Copy link

@dannyadair dannyadair left a comment

Choose a reason for hiding this comment

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

Looks good to me

As per https://github.com/OCA/OpenUpgrade/blob/16.0/openupgrade_scripts/scripts/stock/16.0.1.1/upgrade_analysis.txt#L57

stock / stock.move.line / product_qty (float) : DEL
stock / stock.move.line / product_uom_qty (float) : DEL required
stock / stock.move.line / reserved_qty (float) : NEW isfunction: function, stored
stock / stock.move.line / reserved_uom_qty (float) : NEW required, hasdefault: default

and

stock / stock.picking / move_ids (one2many) : NEW relation: stock.move
stock / stock.picking / move_lines (one2many) : DEL relation: stock.move

@ps-tubtim ps-tubtim force-pushed the 16.0-mig-stock_split_picking branch 2 times, most recently from cd40461 to 02d40d2 Compare December 27, 2022 03:58
AungKoKoLin1997
AungKoKoLin1997 suggested changes Dec 27, 2022
View reviewed changes
ioans73
ioans73 approved these changes Jan 5, 2023
View reviewed changes
Copy link
Member

@ioans73 ioans73 left a comment

Choose a reason for hiding this comment

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

Functional review

@ps-tubtim ps-tubtim force-pushed the 16.0-mig-stock_split_picking branch from c626ca7 to c971755 Compare January 9, 2023 07:11
guewen and others added 17 commits January 9, 2023 14:12
@guewen @ps-tubtim
[MRG] from master branch
44e6392 
(/home/jgrandguillaume/c2c/ProjetsEC/IFRC/Code/openerp_70_ifrc_core_dev/parts/generic-addons rev 29.1.1)
@guewen @ps-tubtim
[FIX] prevents a KeyError is the context is None
73b07dd 
(/home/jgrandguillaume/c2c/ProjetsEC/IFRC/Code/openerp_70_ifrc_core_dev/parts/generic-addons rev 29.1.14)
@nbessi @ps-tubtim
[FIX] donor in manifest
c3a6803
@gurneyalex @ps-tubtim
spellchecking
2256df5
@guewen @ps-tubtim
Display the split button when the delivery order is in draft, waiting…
74ae77d 
… availability or ready to deliver
@gurneyalex @ps-tubtim
mark unported module as uninstallable
82f92e1
@yvaucher @ps-tubtim
Reactivate stock_split_picking
28e5b5b
@yvaucher @ps-tubtim
remove stock.picking.out
99b39ab
@yvaucher @ps-tubtim
Port to 8.0
c7f1684
@yvaucher @ps-tubtim
Adapt tests
9d58496
@guewen @ps-tubtim
Split procurements as well when a move is split
cc9f870 
Both procurements will still be split to the sale order line.
If one of the picking is canceled, the whole sale order will be in
'shipping_exception'.
@guewen @ps-tubtim
Allow to split a partially available picking
8c720f0
@gurneyalex @ps-tubtim
also show split button on incoming moves
07c8365
@gurneyalex @ps-tubtim
handle reservations
2dbedd1 
when splitting a move with reservations, they have to be recomputed.
code inspired from the implementation of stock_picking.rereserve_quants

split the tests in travis.yml because another module is changing the states we get
with only stock_split_picking installed

fix a bug in stock_ownership_by_move which would cause wrong pack operations to
be generated
@sbidoul @ps-tubtim
[UPD] prefix versions with 8.0
200cf6c
@pedrobaeza @ps-tubtim
[MIG] Make modules uninstallable
7a09356
@pedrobaeza @ps-tubtim
[MIG] Rename manifest files
f43c9e3
sbidoul and others added 16 commits January 9, 2023 14:12
@sbidoul @ps-tubtim
remove obsolete .pot files [ci skip]
6de3a3c
@ps-tubtim
[MIG] stock_split_picking: Migration to 12.0
d980c5e
@ps-tubtim
[MIG] stock_split_picking: Pre-commit to 13.0
7e4dc1d
@ps-tubtim
[MIG] stock_split_picking: Migration to 13.0
aaada89
@hbrunn @ps-tubtim
[ADD] stock_split_picking: Mode selection to split in different ways
4919b85
@RujiaL @ps-tubtim
[IMP] stock_split_picking: black, isort, prettier
e96fc79
@RujiaL @ps-tubtim
[MIG] stock_split_picking: Migration to 14.0
a2e4b44
@BernatPForgeFlow @ps-tubtim
[MIG] stock_split_picking: Migration to 15.0
d83ebd9
@ps-tubtim
[UPD] Update stock_split_picking.pot
95c9d55
@OCA-git-bot @ps-tubtim
[UPD] README.rst
8ffce8e
@pedrobaeza @ps-tubtim
[FIX] stock_split_picking: README
02913e9
@JordiMForgeFlow @ps-tubtim
[IMP] stock_split_picking: allow splitting ppickings waiting for othe…
992bc21 
…r operations
@OCA-git-bot @ps-tubtim
[UPD] README.rst
83f952a
@OCA-git-bot @ps-tubtim
stock_split_picking 15.0.1.1.0
3a15a11
@ps-tubtim
[IMP] stock_split_picking: pre-commit stuff
5bbccb9
@ps-tubtim
[MIG] stock_split_picking: Migration to 16.0
88d51b4
@ps-tubtim ps-tubtim force-pushed the 16.0-mig-stock_split_picking branch from c971755 to 88d51b4 Compare January 9, 2023 07:12
@OCA-git-bot
Copy link
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). 🤖

@rousseldenis
Copy link
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 7f2f7dc into OCA:16.0 Jan 24, 2023
@OCA-git-bot
Copy link
Contributor

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

Vicent-S73 pushed a commit to Studio73/e-commerce that referenced this pull request Aug 12, 2025
@ioans73 @fuentes73
build(odoo): OCA/stock-logistics-workflow#1107
0c2e3df
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

+3 more reviewers

@dannyadair dannyadair dannyadair approved these changes

@ioans73 ioans73 ioans73 approved these changes

@AungKoKoLin1997 AungKoKoLin1997 AungKoKoLin1997 approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
approved merged 🎉 migration needs review ready to merge
Projects
None yet
Milestone
16.0
Development

Successfully merging this pull request may close these issues.