Skip to content

Conversation

@ivantodorovich
Copy link
Contributor

By shifting the constraint to the commercial entity, computed from the partner field, we can now select hierarchy contacts.

This is useful to:

A) Register the contact person of the agreement
B) Support modules like partner_affiliate, in case of nested businesses

@OCA-git-bot
Copy link
Contributor

Hi @alexis-via, @ygol,
some modules you are maintaining are being modified, check this out!

@ygol
Copy link
Contributor

ygol commented Aug 18, 2025

LGTM

@ivantodorovich ivantodorovich marked this pull request as ready for review August 18, 2025 15:26
@ivantodorovich
Copy link
Contributor Author

CI is red, but unrelated to this PR.

@ivantodorovich
Copy link
Contributor Author

CI is fixed with this PR, that should be merged first:

By shifting the constraint to the commercial entity, computed from the partner
field, we can now select hierarchy contacts.

This is useful to:

A) Register the contact person of the agreement
B) Support modules like `partner_affiliate`, in case of nested businesses
@ivantodorovich ivantodorovich force-pushed the 18.0-imp-agreement-commercial-partner branch from 03e2ad6 to 7506bd4 Compare September 17, 2025 18:37
@ivantodorovich
Copy link
Contributor Author

rebased and all's green

@leemannd
Copy link

leemannd commented Dec 9, 2025

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

@leemannd The rebase process failed, because command git push --force camptocamp tmp-pr-83:18.0-imp-agreement-commercial-partner failed with output:

remote: Permission to camptocamp/agreement.git denied to OCA-git-bot.
fatal: unable to access 'https://github.com/camptocamp/agreement/': The requested URL returned error: 403

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.

5 participants