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

[13.0][MIG] stock_inventory_exclude_sublocation #817

Merged

Conversation

HviorForgeFlow
Copy link
Member

Migration to v13 tests simplification as well as Inventory Adjustment options have been modified.

CC @ForgeFlow

LoisRForgeFlow and others added 22 commits December 30, 2019 10:13
* [ADD] stock_inventory_exclude_sublocation
It also includes the removal of an unneeded dependency + better position of the field in view
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_inventory_exclude_sublocation
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_inventory_exclude_sublocation/
Currently translated at 100.0% (2 of 2 strings)

Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_inventory_exclude_sublocation
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_inventory_exclude_sublocation/pt_BR/
Currently translated at 100.0% (2 of 2 strings)

Translation: stock-logistics-warehouse-12.0/stock-logistics-warehouse-12.0-stock_inventory_exclude_sublocation
Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-12-0/stock-logistics-warehouse-12-0-stock_inventory_exclude_sublocation/zh_CN/
@max3903 max3903 added this to the 13.0 milestone Dec 31, 2019
Copy link

@AdriaGForgeFlow AdriaGForgeFlow left a comment

Choose a reason for hiding this comment

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

Is there a reason to remove the tests?

Also small comment below

stock_inventory_exclude_sublocation/__manifest__.py Outdated Show resolved Hide resolved
@OCA-git-bot OCA-git-bot mentioned this pull request Jan 3, 2020
54 tasks
Copy link
Sponsor 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. Minor change

stock_inventory_exclude_sublocation/__manifest__.py Outdated Show resolved Hide resolved
@HviorForgeFlow HviorForgeFlow force-pushed the 13.0-mig-stock_inventory_exclude_sublocation branch from 871505f to 4d32950 Compare January 7, 2020 07:17
@HviorForgeFlow
Copy link
Member Author

Is there a reason to remove the tests?

Stock Inventory options has been simplify, that tests was testing old features.

@pedrobaeza does it make sense to you?

@pedrobaeza
Copy link
Member

does it make sense to you?

Yes if the new flows are covered.

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

@LoisRForgeFlow
Copy link
Contributor

Thanks for the work!

/ocabot merge

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 13.0-ocabot-merge-pr-817-by-LoisRForgeFlow-bump-no, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Jan 8, 2020
Signed-off-by LoisRForgeFlow
@OCA-git-bot OCA-git-bot merged commit 4d32950 into OCA:13.0 Jan 8, 2020
@OCA-git-bot
Copy link
Contributor

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

@LoisRForgeFlow LoisRForgeFlow deleted the 13.0-mig-stock_inventory_exclude_sublocation branch January 8, 2020 16:35
CarlosRoca13 pushed a commit to Tecnativa/stock-logistics-warehouse that referenced this pull request Aug 16, 2021
Signed-off-by LoisRForgeFlow
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