Skip to content

[16.0][ADD] project_partner_company_only: Restrict partner to company#1611

Open
maisim wants to merge 1 commit intoOCA:16.0from
maisim:16.0-add-project_partner_id_company_only
Open

[16.0][ADD] project_partner_company_only: Restrict partner to company#1611
maisim wants to merge 1 commit intoOCA:16.0from
maisim:16.0-add-project_partner_id_company_only

Conversation

@maisim
Copy link
Copy Markdown

@maisim maisim commented Nov 27, 2025

Restrict partner_id to partner of type company

Copy link
Copy Markdown
Contributor

@alexey-pelykh alexey-pelykh left a comment

Choose a reason for hiding this comment

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

Thanks for this! Clean and focused module. The domain composition on partner_id covers all the right cases (company flag, non-private type, multi-company). Tests properly verify the view arch via get_view. LGTM

@maisim maisim force-pushed the 16.0-add-project_partner_id_company_only branch from 7c7ba59 to bdfe256 Compare March 24, 2026 17:23
@maisim
Copy link
Copy Markdown
Author

maisim commented Mar 24, 2026

@alexey-pelykh
I made the same changes as in OCA/sale-workflow#4042

  • Rename the module
  • Add negative tests
  • Fix copyright
  • use base_view_inheritance_extension

@maisim maisim force-pushed the 16.0-add-project_partner_id_company_only branch from bdfe256 to 568cebf Compare March 24, 2026 17:29
@maisim maisim changed the title [16.0][ADD] project_partner_id_company_only: Restrict partner to company [16.0][ADD] project_partner_company_only: Restrict partner to company Mar 25, 2026
maisim added a commit to maisim/sale-workflow that referenced this pull request Mar 25, 2026
…unctional tests from runboat

TODO: remove once the following PRs are merged:
- OCA#4042 (sale_order_partner_company_only)
- OCA/project#1611 (project_partner_company_only)
- OCA/account-invoicing#2180 (account_invoice_partner_company_only)
maisim added a commit to maisim/sale-workflow that referenced this pull request Mar 25, 2026
…unctional tests from runboat

TODO: remove once the following PRs are merged:
- OCA#4042 (sale_order_partner_company_only)
- OCA/project#1611 (project_partner_company_only)
- OCA/account-invoicing#2180 (account_invoice_partner_company_only)
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.

3 participants