-
-
Notifications
You must be signed in to change notification settings - Fork 697
[17.0][MIG] mail_gateway_whatsapp: Migration to 17.0 #1619
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
[17.0][MIG] mail_gateway_whatsapp: Migration to 17.0 #1619
Conversation
37ed58a to
81a577e
Compare
|
/ocabot migration mail_gateway_whatsapp @CarlosRoca13 please rebase removing temporary commit. |
Allow to send documents, images and so on add mark on message Improve security Pass logic to components Add tests Manage change of version of Facebook API
Currently translated at 4.1% (4 of 96 strings) Translation: social-16.0/social-16.0-mail_gateway_whatsapp Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_gateway_whatsapp/es/
If the company wants to initiate a conversation with a customer, a template must be used; otherwise, messages will not be sent. This also applies when the last conversation with the customer is older than 24 hours. - Enabled downloading templates from META - Added the ability to create templates directly in Odoo. Note: Currently, templates with variables or buttons are not supported.
…ated to the current user - Use OdooBot for webhooks instead of the current user, following the same logic as Odoo. - Avoid automatically adding the current user as a member of a new gateway to prevent notifications for all messages. This is now an explicit configuration.
Currently translated at 100.0% (210 of 210 strings) Translation: social-16.0/social-16.0-mail_gateway_whatsapp Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_gateway_whatsapp/it/
ea9bb16 to
0e6b019
Compare
|
Done @pedrobaeza |
carlos-lopez-tecnativa
left a comment
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.
Tested locally and it's working, just a minor change.
|
|
||
| def _generate_avatar_gateway(self): | ||
| if self.gateway_id.gateway_type == "whatsapp": | ||
| path = get_resource_path( |
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 method is deprecated. Could you please update it to use file_path instead?
https://github.com/odoo/odoo/blob/16718e2b0b3fa8acb4827477f26da22f74397cd3/odoo/modules/module.py#L191-L203
0e6b019 to
c69bced
Compare
carlos-lopez-tecnativa
left a comment
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
|
/ocabot merge nobump |
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at eab7e7e. Thanks a lot for contributing to OCA. ❤️ |
Depends on
cc @Tecnativa TT50730
@carlos-lopez-tecnativa can you review please?