-
-
Notifications
You must be signed in to change notification settings - Fork 281
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
[16.0][IMP] mail_multicompany: search most appropriate server based on the from email #612
[16.0][IMP] mail_multicompany: search most appropriate server based on the from email #612
Conversation
636faf5
to
8a970f3
Compare
Hi @luisg123v, |
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.
code review
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
self.sudo().env["ir.mail_server"].search([("company_id", "=", company_id)]) | ||
) | ||
if not mail_servers: | ||
return False, email_from |
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.
return False, email_from | |
return None, email_from |
as the same odoo does in _find_mail_server
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.
Missing commit description.
8a970f3
to
87541e5
Compare
This commit improves the create() overwritte on the mail_message class so that the mail server selected is the most apropriate taking into account the from email, but considering only the mail servers of the current company.
87541e5
to
55b5d47
Compare
/ocabot merge minor |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 6dd0d41. Thanks a lot for contributing to OCA. ❤️ |
No description provided.