Skip to content

[ADD] partner_owner_company: soft alternative to company_id#2271

Open
dreispt wants to merge 1 commit intoOCA:17.0from
ursais:17-dr-partner_owner_company
Open

[ADD] partner_owner_company: soft alternative to company_id#2271
dreispt wants to merge 1 commit intoOCA:17.0from
ursais:17-dr-partner_owner_company

Conversation

@dreispt
Copy link
Member

@dreispt dreispt commented Jan 21, 2026

No description provided.

@dreispt dreispt force-pushed the 17-dr-partner_owner_company branch from 1343d97 to 564ad38 Compare February 6, 2026 19:25
Copy link

@luisDIXMIT luisDIXMIT left a comment

Choose a reason for hiding this comment

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

LGTM but pre-commit is failing.

<xpath expr="//field[@name='category_id']" position="after">
<field
name="owner_company_id"
options="{'no_create': True, 'no_create_edit': True}"

Choose a reason for hiding this comment

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

I think if it is a company, it should be invisible, following the logic in the tree view.

Choose a reason for hiding this comment

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

Hi @dreispt, don’t you think so?

@dreispt dreispt force-pushed the 17-dr-partner_owner_company branch from 564ad38 to 638d82f Compare February 26, 2026 22:08
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.

2 participants