-
-
Notifications
You must be signed in to change notification settings - Fork 636
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
[ADD] mail_contact_type: #1264
[ADD] mail_contact_type: #1264
Conversation
a6ea5dd
to
876ea51
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
e4307f2
to
8379a38
Compare
1cbd2a5
to
d97aef0
Compare
name = super()._get_name() | ||
|
||
if ( | ||
not self._context.get("show_mail_contact_types") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@alexandregaldeano, @barkat-matthias I can't see where this context is used in this module ?
I suppose we should add it in the mail composer (not the one from accounting to avoid extra dependency which should be take care by an other module.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cd9f71c
to
a261bc7
Compare
Provides a method to use on mail templates to select contacts by their categories. --------- Co-authored-by: Pierre Verkest <[email protected]> Co-authored-by: Alexandre Galdeano <[email protected]> Co-authored-by: Matthias BARKAT <[email protected]>
a261bc7
to
0e290a8
Compare
…contact-selection [IMP] - Better contact selection by adding & and or condition
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't see anything wrong with the code, but if I were to implement such a functionality I'd build on partner_multi_relation, add a code field to relation types and look at that code in contact_by_types
/_name_search
name = super()._get_name() | ||
|
||
if ( | ||
not self._context.get("show_mail_contact_types") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR has the |
/ocabot merge nobump |
On my way to merge this fine PR! |
Congratulations, your PR was merged at dd253c8. Thanks a lot for contributing to OCA. ❤️ |
Provides a method to use on mail templates to select contacts by their categories.