-
-
Notifications
You must be signed in to change notification settings - Fork 699
[18.0][MIG] mail_gateway_whatsapp: Migration to 18.0 #1703
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
[18.0][MIG] mail_gateway_whatsapp: Migration to 18.0 #1703
Conversation
|
/ocabot migration mail_gateway_whatsapp |
4de4454 to
d152729
Compare
82e7b3c to
d3555bb
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.
Tested locally with #1701
LGTM, just a minor change.
| <?xml version="1.0" encoding="UTF-8" ?> | ||
| <templates xml:space="preserve"> | ||
|
|
||
| <t t-name="mail_gateway_whatsapp.SendWhatsappButton" owl="1"> |
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.
| <t t-name="mail_gateway_whatsapp.SendWhatsappButton" owl="1"> | |
| <t t-name="mail_gateway_whatsapp.SendWhatsappButton"> |
| selection_add=[("whatsapp", "WhatsApp")], ondelete={"whatsapp": "cascade"} | ||
| ) | ||
| whatsapp_from_phone = fields.Char() | ||
| whatsapp_version = fields.Char(default="15.0") |
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.
The minimal API version supported by META for new apps is 23.0. It could be a good time to update this as the default value. WDYT?
| whatsapp_version = fields.Char(default="15.0") | |
| whatsapp_version = fields.Char(default="23.0") |
d3555bb to
dc6b288
Compare
|
Changes done @carlos-lopez-tecnativa |
|
Remove temp 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/
dc6b288 to
d933a99
Compare
|
Done @pedrobaeza |
|
I'll wait till @carlos-lopez-tecnativa approval, as I didn't see the one in the dependency. |
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
pedrobaeza
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.
/ocabot merge nobump
|
On my way to merge this fine PR! |
|
Congratulations, your PR was merged at b3dc417. Thanks a lot for contributing to OCA. ❤️ |
cc @Tecnativa TT57123
This PR depends on: