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

[16.0][IMP] stock_inventory: various fixes #2029

Merged
merged 2 commits into from
May 16, 2024

Conversation

DavidJForgeFlow
Copy link
Contributor

Forward port of #1991 #2010.

Supersedes #2017

@ForgeFlow

DavidJForgeFlow and others added 2 commits May 13, 2024 13:07
Allow to create stock.quants making the quants smart button not invisible when quantity is equal to 0. (Removing one of the invisible conditions).

[IMP] stock_inventory: adds exclude_sublocation flag

[IMP] stock_inventory: Add flag to autocomplete adjustment when fully done.

[IMP] stock_inventory

Add company to adjustments to avoid multi-company errors. Adds information fields to tree view. Also adds 'Assign to' in stock inventory that propagates to quants. Also propagates the date field.
Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

Ty

@LoisRForgeFlow
Copy link
Contributor

I'm merging to align with v15.

/ocabot merge major

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-2029-by-LoisRForgeFlow-bump-major, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 3378a5e into OCA:16.0 May 16, 2024
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

Comment on lines -153 to +166
<delete model="ir.ui.menu" id="stock.menu_action_inventory_tree" />
<record id="stock.menu_action_inventory_tree" model="ir.ui.menu">
<field name="active" eval="False" />
</record>
Copy link

@Kev-Roche Kev-Roche May 17, 2024

Choose a reason for hiding this comment

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

If we already have the module installed, the view does not exist anymore, so we can not access to "stock.menu_action_inventory_tree" and an error occurs.
@DavidJForgeFlow @LoisRForgeFlow

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi! You can just update the module "stock" and then install / update the "stock_inventory" module!

Choose a reason for hiding this comment

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

Of course, but depending of the odoo hosting infrastructure, it is not a suitable / clean solution. Returning to the previous line of code or a small migration script will work without effort. what do you think ?

Copy link

Choose a reason for hiding this comment

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

"You can just update the module "stock" and then install / update the "stock_inventory" module!" - I confirm this works but it's not intuitive or documented anywhere except in this thread. A better solution needs to be found. Is it possible to force the upgrade of "stock" before upgrading this module?

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.

6 participants