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

[17.0][MIG] stock_reserve: Migration to 17.0 #1934

Merged
merged 76 commits into from
Mar 20, 2024
Merged

[17.0][MIG] stock_reserve: Migration to 17.0 #1934

merged 76 commits into from
Mar 20, 2024

Conversation

chrisandrewmann
Copy link

No description provided.

@OCA-git-bot
Copy link
Contributor

Hi @bodedra,
some modules you are maintaining are being modified, check this out!

@chrisandrewmann chrisandrewmann mentioned this pull request Feb 7, 2024
43 tasks
@chrisandrewmann
Copy link
Author

Hi @bodedra and @pedrobaeza
Any chance of getting this merged pleased?

@pedrobaeza
Copy link
Member

/ocabot migration stock_reserve

It should be reviewed by a PSC or module maintainer for being merged.

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Mar 14, 2024
Copy link
Sponsor Contributor

@celm1990 celm1990 left a comment

Choose a reason for hiding this comment

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

Please squash commits

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

@chrisandrewmann
Copy link
Author

Please squash commits

@celm1990 Sorry not sure which commits you mean, I thought i'd followed the procecdure.
Also not sure why codecov/project is failing, looking at the analysis it's referring to files i've not even touched.

guewen and others added 19 commits March 15, 2024 08:54
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.
In order to get visibility on https://www.odoo.com/apps the OCA board has
decided to add the OCA as author of all the addons maintained as part of the
association.
This has always been wrong on v8, but because of the very obscure
odoo/odoo#5797, this seemed to work every time we had single-step
reception.

Please note that this block of XML is noupdate, so for existing
installations you need to change the parent of the reservation location
to something outside your warehouses manually.

Incidentally, this makes the branch green independently of odoo/odoo#5797.
francesco-ooops and others added 11 commits March 15, 2024 08:54
Currently translated at 17.6% (27 of 153 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_reserve/it/
Currently translated at 100.0% (153 of 153 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_reserve/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_reserve/
Currently translated at 100.0% (155 of 155 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_reserve/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_reserve/
Currently translated at 100.0% (156 of 156 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_reserve/es/
@rousseldenis
Copy link
Sponsor Contributor

Sorry not sure which commits you mean, I thought i'd followed the procecdure.

No, that's ok like that.

mymage and others added 9 commits March 15, 2024 09:06
Currently translated at 17.3% (27 of 156 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_reserve/it/
Currently translated at 100.0% (156 of 156 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_reserve/es/
Currently translated at 100.0% (156 of 156 strings)

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_reserve/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: stock-logistics-warehouse-16.0/stock-logistics-warehouse-16.0-stock_reserve
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_reserve/
@chrisandrewmann
Copy link
Author

chrisandrewmann commented Mar 15, 2024

I've rebased with 17.0 and then squashed any duplicate "weblate" italian translations to clean it up a bit.
Hope this works ok.
Update: Still failing codecov/project. @rousseldenis any idea why this is?

@dreispt
Copy link
Sponsor Member

dreispt commented Mar 20, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 17.0-ocabot-merge-pr-1934-by-dreispt-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 48f934b into OCA:17.0 Mar 20, 2024
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@chrisandrewmann chrisandrewmann deleted the 17.0-mig-stock_reserve branch March 20, 2024 15:25
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.

None yet