Skip to content

[17.0][MIG] sale stock warehouse multicompany#915

Open
weinni2000 wants to merge 7 commits intoOCA:17.0from
weinni2000:17.0-mig-sale_stock_warehouse_multicompany
Open

[17.0][MIG] sale stock warehouse multicompany#915
weinni2000 wants to merge 7 commits intoOCA:17.0from
weinni2000:17.0-mig-sale_stock_warehouse_multicompany

Conversation

@weinni2000
Copy link

No description provided.

@weinni2000 weinni2000 force-pushed the 17.0-mig-sale_stock_warehouse_multicompany branch 4 times, most recently from ff74f46 to aeb393e Compare December 4, 2025 12:08
"license": "AGPL-3",
"application": False,
"installable": True,
"depends": ["delivery"],

Choose a reason for hiding this comment

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

Tested with a fresh clean install locally, the dependency for stock is still needed in 17.0 (Install fails with TypeError: Model 'stock.warehouse' does not exist in registry. in a new DB)

Copy link
Author

Choose a reason for hiding this comment

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

@cyrilmanuel & @mymage :
I see you are parts of this repo. Could you have a look at the PR and approve and merge?

@weinni2000 weinni2000 force-pushed the 17.0-mig-sale_stock_warehouse_multicompany branch from aeb393e to 999d7e8 Compare December 4, 2025 15:00
@weinni2000 weinni2000 requested a review from ap-wtioit December 4, 2025 17:01
@ap-wtioit
Copy link

LGTM, tested locally and on runboat

Copy link

@it-fact4lifetimesupport it-fact4lifetimesupport left a comment

Choose a reason for hiding this comment

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

LGTM

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

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

Comments