Skip to content

[19.0][MIG] stock_inventory_analytic_warehouse_default - #931

Open
glitchov wants to merge 5 commits into
OCA:19.0from
acsone:19.0-mig-stock_inventory_analytic_warehouse_default-hda
Open

[19.0][MIG] stock_inventory_analytic_warehouse_default#931
glitchov wants to merge 5 commits into
OCA:19.0from
acsone:19.0-mig-stock_inventory_analytic_warehouse_default-hda

Conversation

@glitchov

@glitchov glitchov commented Jun 8, 2026

Copy link
Copy Markdown

Migration from 14.0
Depends on #929 and #930

rousseldenis and others added 4 commits June 8, 2026 09:13
@OCA-git-bot OCA-git-bot added series:19.0 mod:stock_inventory_analytic_warehouse_default Module stock_inventory_analytic_warehouse_default labels Jun 8, 2026
@glitchov
glitchov force-pushed the 19.0-mig-stock_inventory_analytic_warehouse_default-hda branch from a3af6e1 to 2857fc7 Compare June 8, 2026 11:06

@FrancoMaxime FrancoMaxime left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM: code review

One typo,


def test_inventory_adjustment_analytic_default(self):
"""Default is still working if no warehouse analytic is provided.
Create a quant and apply physical inventory to it hen check that the

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
Create a quant and apply physical inventory to it hen check that the
Create a quant and apply physical inventory to it then check that the

@BhaveshHeliconia BhaveshHeliconia left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Functional review LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved mod:stock_inventory_analytic_warehouse_default Module stock_inventory_analytic_warehouse_default series:19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants