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][ADD]sale_order_warehouse_location: Set warehouse in sale order based on delivery country and state #3592

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

manuelregidor
Copy link
Contributor

@manuelregidor manuelregidor commented Feb 25, 2025

New module to automatically set the warehouse in a sale order depending on the country and state of the delivery address.

An error in tests seems to be caused by these changes in account-invoicing repository: OCA/account-invoicing#1840

@HaraldPanten @ValentinVinagre @luis-ron

T-7589

Copy link
Contributor

@ValentinVinagre ValentinVinagre left a comment

Choose a reason for hiding this comment

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

some comments

Copy link

@luis-ron luis-ron left a comment

Choose a reason for hiding this comment

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

Functional Review: LGTM 👍🏻

@manuelregidor manuelregidor changed the title [16.0][ADD]sale_order_warehouse_location: Set warehouse is sale order based on delivery country and state [16.0][ADD]sale_order_warehouse_location: Set warehouse in sale order based on delivery country and state Feb 28, 2025
Copy link

@Tisho99 Tisho99 left a comment

Choose a reason for hiding this comment

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

LGTM

@manuelregidor manuelregidor force-pushed the 16.0-add-sale_order_warehouse_location branch from 3db6aff to 8478f1d Compare February 28, 2025 09:56
Copy link
Contributor

@ValentinVinagre ValentinVinagre left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants