Skip to content
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

[IMP] sms: Fallback on partner if records don't inherit mail.thread #71020

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

qmo-odoo
Copy link
Contributor

Before this commit, trying to use the sms composer on a record that is not inheriting
mail.thread and has a phone field that is related to a partner would be considered invalid
and users would have to retype the number themselves.

After this commit, if the sms composer is used one a phone field from a record that does not
inherit mail.thread but has a partner_id, it will use the related partner mail.thread abilities
to get the recipients info instead of just considering the number to be invalid.

LINKS:
TaskID: 2514622
PR: #xxxxx

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@qmo-odoo qmo-odoo requested review from a team as code owners May 19, 2021 10:04
@robodoo
Copy link
Contributor

robodoo commented May 19, 2021

@qmo-odoo qmo-odoo marked this pull request as draft May 19, 2021 10:23
@C3POdoo C3POdoo added the RD research & development, internal work label May 19, 2021
Before this commit, trying to use the sms composer on a record that is not inheriting
mail.thread and has a  phone field that is related to a partner would be considered invalid
and users would have to retype the number themselves.

After this commit, if the sms composer is used one a phone field from a record that does not
inherit mail.thread but has a partner_id, it  will use the related partner mail.thread abilities
to get the recipients info instead of just considering the number to be invalid.

LINKS:
TaskID: 2514622
PR: #xxxxx
@qmo-odoo qmo-odoo force-pushed the master-sms-composer-on-related-fields-qmo branch from a07c2e5 to e7f9f94 Compare June 11, 2021 06:36
@seb-odoo seb-odoo removed the request for review from a team July 6, 2023 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants