-
-
Notifications
You must be signed in to change notification settings - Fork 720
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
[15.0][MIG] stock_cycle_count #1995
Conversation
220a89d
to
fe299c8
Compare
/ocabot migration stock_cycle_count |
@ArnauCForgeFlow To help review, don't hesitate to put depending PR's links in this PR description. Moreover, you can add it through : https://github.com/OCA/maintainer-tools/wiki/Use-temporary-reference%28s%29-to-another-pull-request%28s%29 In order to make tests running and be able to launch runboat |
stock_inventory_exclude_sublocation is no longer a dependency after the change done in #1991 |
* Update README. * Fixes: DEFAULT_SERVER_DATETIME_FORMAT, PERCENT variables and sale price calculation.
…manual creation of cycle counts.
…racking to some of them.
…tment related to a cycle count cannot be modified.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: stock-logistics-warehouse-14.0/stock-logistics-warehouse-14.0-stock_cycle_count Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-14-0/stock-logistics-warehouse-14-0-stock_cycle_count/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: stock-logistics-warehouse-14.0/stock-logistics-warehouse-14.0-stock_cycle_count Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-14-0/stock-logistics-warehouse-14-0-stock_cycle_count/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: stock-logistics-warehouse-14.0/stock-logistics-warehouse-14.0-stock_cycle_count Translate-URL: https://translation.odoo-community.org/projects/stock-logistics-warehouse-14-0/stock-logistics-warehouse-14-0-stock_cycle_count/
e9694d3
to
1f0c9ab
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As a strict migration, I think this one is good to go.
After this, more improvements to come in #1997.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functional review! LGTM!
This PR has the |
@ArnauCForgeFlow litte detail Remove the duplicated Assigned to from views |
1f0c9ab
to
7b282e1
Compare
@LoisRForgeFlow Done! also removed company_id field since it is now declared in stock_inventory module. |
Thanks! /ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at baeb889. Thanks a lot for contributing to OCA. ❤️ |
Migration to 15.0