Skip to content

Conversation

@chienandalu
Copy link
Member

@chienandalu chienandalu commented Mar 18, 2025

@chienandalu chienandalu force-pushed the 17.0-mig-mail_gateway_whatsapp branch 6 times, most recently from 37ed58a to 81a577e Compare March 19, 2025 11:20
@chienandalu chienandalu marked this pull request as ready for review March 19, 2025 12:28
@pedrobaeza
Copy link
Member

/ocabot migration mail_gateway_whatsapp

@CarlosRoca13 please rebase removing temporary commit.

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Jul 22, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Jul 22, 2025
40 tasks
olgamarcocb and others added 13 commits July 23, 2025 06:41
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/
@CarlosRoca13 CarlosRoca13 force-pushed the 17.0-mig-mail_gateway_whatsapp branch from ea9bb16 to 0e6b019 Compare July 23, 2025 04:43
@CarlosRoca13
Copy link
Contributor

Done @pedrobaeza

Copy link
Contributor

@carlos-lopez-tecnativa carlos-lopez-tecnativa left a 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(
Copy link
Contributor

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

@CarlosRoca13 CarlosRoca13 force-pushed the 17.0-mig-mail_gateway_whatsapp branch from 0e6b019 to c69bced Compare July 23, 2025 11:53
@CarlosRoca13
Copy link
Contributor

Done @carlos-lopez-tecnativa

Copy link
Contributor

@carlos-lopez-tecnativa carlos-lopez-tecnativa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pedrobaeza pedrobaeza changed the title [MIG] mail_gateway_whatsapp: Migration to 17.0 [17.0][MIG] mail_gateway_whatsapp: Migration to 17.0 Jul 23, 2025
@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-1619-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 18b25c5 into OCA:17.0 Jul 23, 2025
8 of 9 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at eab7e7e. Thanks a lot for contributing to OCA. ❤️

@pedrobaeza pedrobaeza deleted the 17.0-mig-mail_gateway_whatsapp branch July 23, 2025 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.