Skip to content
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.

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

[MIG] stock_reserve 15.0 #1485

Merged
merged 35 commits into from
Nov 29, 2022
Merged

Conversation

flachica
Copy link
Contributor

No description provided.

@flachica flachica mentioned this pull request Sep 15, 2022
80 tasks
@flachica flachica force-pushed the 15.0-mig-stock_reserve branch 2 times, most recently from 624efd4 to 303302a Compare September 15, 2022 10:54
@oca-clabot
Copy link

Hey @flachica, 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

@victoralmau
Copy link
Member

Thank you for your work.
There is something wrong with this migration, the stock_reserve_sale addon is added and it should not.

Maybe it would be a good idea to finish the migration to v14 (#1346) and then continue with this one.

@flachica
Copy link
Contributor Author

Sorry @victoralmau, this module is migrated in #1486. I got confused with branch

@rousseldenis
Copy link
Contributor

/ocabot migration stock_reserve

@rousseldenis
Copy link
Contributor

@flachica Could you remove unrelated files (stock_reserve_sale) ?

@flachica
Copy link
Contributor Author

flachica commented Oct 7, 2022

@flachica Could you remove unrelated files (stock_reserve_sale) ?

@rousseldenis In this PR I got confused about the branch but I thought I had solved it. Please, can you tell me where you see the remaining files? I can not find it

@rousseldenis
Copy link
Contributor

@flachica Could you remove unrelated files (stock_reserve_sale) ?

@rousseldenis In this PR I got confused about the branch but I thought I had solved it. Please, can you tell me where you see the remaining files? I can not find it

No, it's ok, wrong browser tab 😓

stock_reserve/model/stock_reserve.py Outdated Show resolved Hide resolved
stock_reserve/model/product.py Outdated Show resolved Hide resolved
stock_reserve/model/product.py Outdated Show resolved Hide resolved
stock_reserve/tests/test_stock_reserve.py Show resolved Hide resolved
stock_reserve/tests/test_stock_reserve.py Outdated Show resolved Hide resolved
stock_reserve/tests/test_stock_reserve.py Outdated Show resolved Hide resolved
stock_reserve/tests/test_stock_reserve.py Outdated Show resolved Hide resolved
stock_reserve/tests/test_stock_reserve.py Outdated Show resolved Hide resolved
guewen and others added 9 commits October 27, 2022 08:28
don't use half the quantity so that the reserved qty is different from
available qty
For weird reasons, force-assigning a move does not associate quants.
action_assign'ing the picking does, and is the same procedure that you
can do via the interface.
[UPG] Upgraded to version 8, fixed references to new 8.0  views and  moved fields that were in  product.product to product.template

[fix] remove duplicate view and correct view name

[UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0.

[UPG] stock available immediately, corrected the calculation method of
immediately_usable_qty to take in accountthe sign change in outgoing_qty
(from negative to positive) in version 8.

[FLAKE8]

[FIX] renaming of a class, comment removing, useless code.

[UPD] move out from unported to 8 for update

[fix] remove duplicate view and correct view name

[UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0.

[FIX] renaming of a class, comment removing, useless code.

[fix] remove duplicate view and correct view name

[UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0.

[UPD] move out from unported to 8 for update

[fix] remove duplicate view and correct view name

[UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0.

[FIX] renaming of a class, comment removing, useless code.

[UPD] move out from unported to 8 for update

[fix] remove duplicate view and correct view name

[UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0.

[FIX] renaming of a class, comment removing, useless code.

[UPD] move out from unported to 8 for update

[fix] remove duplicate view and correct view name

[UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0.

[FIX] renaming of a class, comment removing, useless code.

[fix] remove duplicate view and correct view name

[UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0.

[UPD] move out from unported to 8 for update

[fix] remove duplicate view and correct view name

[UPG][FIX] added outgoing field XML, that was in the base stock field in 7.0.

[FIX] renaming of a class, comment removing, useless code.
@flachica
Copy link
Contributor Author

@victoralmau How I can set this commit as coauthored with you? You are made a lot of valueable suggestions

- Additionally include note field in stock reservation. It [was](odoo/odoo@71203e0) removed
@rousseldenis
Copy link
Contributor

rousseldenis commented Oct 27, 2022

@victoralmau How I can set this commit as coauthored with you? You are made a lot of valueable suggestions

You can use git cherry-pick to do so. Or oca-port tool https://pypi.org/project/oca-port/

Copy link
Member

@victoralmau victoralmau left a comment

Choose a reason for hiding this comment

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

Code review OK.

IMO, you don't need to add me as a co-author.

@CasVissers-360ERP
Copy link

@rousseldenis Think this one should be ready to merge?

Copy link
Contributor

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

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

Code review

@rousseldenis
Copy link
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 15.0-ocabot-merge-pr-1485-by-rousseldenis-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 1cbe2ee into OCA:15.0 Nov 29, 2022
@OCA-git-bot
Copy link
Contributor

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