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

[17.0][ADD] sale_order_restrict_customer_field #86

Draft
wants to merge 1 commit into
base: 17.0
Choose a base branch
from

Conversation

IriaAlonso
Copy link
Contributor

@IriaAlonso IriaAlonso commented Nov 29, 2024

  • Add is_customer field on res.partner that indicates if a partner is customer or delivery address
  • Add domain to partner_id bassed on is_customer

@IriaAlonso IriaAlonso force-pushed the 17.0-add-sale_order_restrict_customer_field branch from f3a0224 to 2726d92 Compare November 29, 2024 09:24
@IriaAlonso
Copy link
Contributor Author

@dalonsod can you review it?

@IriaAlonso IriaAlonso changed the title [17.0][ADD] sale_order_restrict_customer_field [WIP][17.0][ADD] sale_order_restrict_customer_field Nov 29, 2024
@IriaAlonso IriaAlonso force-pushed the 17.0-add-sale_order_restrict_customer_field branch 2 times, most recently from c63d8a7 to 08348fd Compare November 29, 2024 11:02
@IriaAlonso
Copy link
Contributor Author

IriaAlonso commented Nov 29, 2024

is_customer field would be True if:

  • type == delivery
  • type == contact and ((child_ids.type == delivery) == 0)

@IriaAlonso IriaAlonso force-pushed the 17.0-add-sale_order_restrict_customer_field branch 2 times, most recently from 92b1f27 to c10f264 Compare November 29, 2024 11:25
@IriaAlonso IriaAlonso changed the title [WIP][17.0][ADD] sale_order_restrict_customer_field [17.0][ADD] sale_order_restrict_customer_field Nov 29, 2024
@IriaAlonso IriaAlonso force-pushed the 17.0-add-sale_order_restrict_customer_field branch from c10f264 to 0e012f8 Compare November 29, 2024 11:56
@IriaAlonso IriaAlonso force-pushed the 17.0-add-sale_order_restrict_customer_field branch from 50f4d3a to c223f53 Compare December 2, 2024 07:38
@IriaAlonso
Copy link
Contributor Author

@dalonsod can you test it?

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

Successfully merging this pull request may close these issues.

1 participant