[17.0] mail_gateway: Migration to 17.0 #1614

chienandalu · 2025-03-05T15:47:28Z

What's been done:

In the client side a big refactor was made by Odoo and it was necessary to refactor the client logic as well for this module stuff.
Aside from the regular migration stuff, some logic had to be adapted the new client stuff.

please review @carlos-lopez-tecnativa @CarlosRoca13

WIP for test workflows of mail_gateway migration in OCA#1614

Currently translated at 100.0% (85 of 85 strings) Translation: social-16.0/social-16.0-mail_gateway Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_gateway/it/

…ed properly

Other OCA modules extending the composer add m2m fields with fixed relations. To avoid a glue module we try for the moment to add those manually. TT51108

Currently translated at 81.2% (113 of 139 strings) Translation: social-16.0/social-16.0-mail_gateway Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_gateway/it/

Currently translated at 10.7% (15 of 139 strings) Translation: social-16.0/social-16.0-mail_gateway Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_gateway/es/

…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.

carlos-lopez-tecnativa

Code review, just a minor changes.
I tested the function locally with mail_gateway_whatsapp, and it worked fine.

carlos-lopez-tecnativa · 2025-03-19T16:20:07Z

mail_gateway/security/security.xml

-    <record id="mail_channel_gateway_rule" model="ir.rule">
-        <field name="name">Mail.channel: access gateway</field>
-        <field name="model_id" ref="mail.model_mail_channel" />
+    <record id="discuss_channel_gateway_rule" model="ir.rule">


Is it really necessary to change the xml_id? This change might require a script for OpenUpgrade, or did I miss something?

Well, is a noupdate="0" record in this case, so the old one will be dropped.

mail_gateway/__manifest__.py

mail_gateway/models/discuss_channel.py

mail_gateway/models/mail_thread.py

mail_gateway/security/security.xml

mail_gateway/static/src/components/chatter/chatter.xml

mail_gateway/views/mail_gateway.xml

carlos-lopez-tecnativa · 2025-03-19T17:30:34Z

mail_gateway/wizards/mail_guest_manage.py

+    def create_partner(self):
+        partner = self.env["res.partner"].create(self._get_partner_vals())
+        self._merge_partner(partner)
+        return partner.get_formview_action()
+
+    def _get_partner_vals(self):
+        return {
+            "name": self.guest_id.name,
+        }


When a new partner is created from an anonymous record, only the name field is set. Is it possible to populate additional fields, such as phone or mobile?

Where do you get them from? In telegram for instance you don't have such data. Maybe in whatsapp it could be inferred from the token...

I don't know if the data is available. I only see that the new contact only has the name set, but the main information for WhatsApp should be the phone number.

This wizard is opened from the chatter when there is no res.partner associated with the guest. Clicking on the name displays the popup.

Yeah, but I mean that should be dealt in mail_gateway_whatsapp, as other gateways might not have that info at all

Yeah, but I mean that should be dealt in mail_gateway_whatsapp, as other gateways might not have that info at all

Ahh, ok, sorry! But I agree, this information is specific to WhatsApp.

TT50728

CarlosRoca13 · 2025-07-21T05:25:58Z

please @carlos-lopez-tecnativa review this again

carlos-lopez-tecnativa

Tested locally. LGTM.

pedrobaeza · 2025-07-22T13:28:09Z

/ocabot migration mail_gateway
/ocabot merge nobump

OCA-git-bot · 2025-07-22T13:28:19Z

On my way to merge this fine PR!
Prepared branch 17.0-ocabot-merge-pr-1614-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot · 2025-07-22T13:41:09Z

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

WIP for test workflows of mail_gateway migration in OCA#1614

chienandalu mentioned this pull request Mar 5, 2025

[17.0][MIG] mail_gateway: Migration to version 17.0 #1613

Closed

chienandalu added a commit to Tecnativa/social that referenced this pull request Mar 5, 2025

[MIG] mail_gategay_telegram: Migration to v17

3320688

WIP for test workflows of mail_gateway migration in OCA#1614

chienandalu mentioned this pull request Mar 5, 2025

[17.0][MIG] mail_gateway_telegram: Migration to v17 #1615

Open

chienandalu force-pushed the 17.0-MIG-mail_gateway branch from 40b0825 to c5ea53f Compare March 14, 2025 11:19

chienandalu added a commit to Tecnativa/social that referenced this pull request Mar 14, 2025

[MIG] mail_gategay_telegram: Migration to v17

74be63d

WIP for test workflows of mail_gateway migration in OCA#1614

chienandalu force-pushed the 17.0-MIG-mail_gateway branch 7 times, most recently from 2960391 to 4116f81 Compare March 18, 2025 10:46

chienandalu added a commit to Tecnativa/social that referenced this pull request Mar 18, 2025

[MIG] mail_gategay_telegram: Migration to v17

2c91203

WIP for test workflows of mail_gateway migration in OCA#1614

chienandalu force-pushed the 17.0-MIG-mail_gateway branch 2 times, most recently from daf2808 to 3c05c03 Compare March 18, 2025 11:07

chienandalu mentioned this pull request Mar 18, 2025

[17.0][MIG] mail_gateway_whatsapp: Migration to 17.0 #1619

Merged

chienandalu force-pushed the 17.0-MIG-mail_gateway branch 2 times, most recently from f22ff5b to a5cfe3a Compare March 18, 2025 16:34

olgamarcocb and others added 12 commits March 19, 2025 12:52

[ADD] mail_broker

6681ddd

[IMP] mail_broker: Set webhook

de68355

[IMP] mail_broker

b448237

[IMP] mail_broker: Black, isort, prettier

de997e2

[MIG] mail_broker: Migration to 14.0

95b93b3

[IMP] mail_broker: Black, isort, prettier

ef1dc51

[MIG] mail_gateway: Migration to 16.0

8b1af36

Added translation using Weblate (Italian)

4f5fbcb

Added translation using Weblate (Spanish)

50b8037

Translated using Weblate (Italian)

315a20a

Currently translated at 100.0% (85 of 85 strings) Translation: social-16.0/social-16.0-mail_gateway Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_gateway/it/

[FIX] mail_gateway: Pass company in order to ensure that it is execut…

dc91075

…ed properly

[IMP] mail_gateway: Allow usage of mail templates on gateway

d68ac28

chienandalu and others added 5 commits March 19, 2025 12:54

[FIX] mail_gateway: dummy compatibility with m2m fields

b5f90da

Other OCA modules extending the composer add m2m fields with fixed relations. To avoid a glue module we try for the moment to add those manually. TT51108

Translated using Weblate (Italian)

0d983ae

Currently translated at 81.2% (113 of 139 strings) Translation: social-16.0/social-16.0-mail_gateway Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_gateway/it/

Translated using Weblate (Spanish)

6b7f980

Currently translated at 10.7% (15 of 139 strings) Translation: social-16.0/social-16.0-mail_gateway Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_gateway/es/

[IMP] mail_gateway: pre-commit auto fixes

67387f3

chienandalu force-pushed the 17.0-MIG-mail_gateway branch from a5cfe3a to 993561c Compare March 19, 2025 12:07

chienandalu marked this pull request as ready for review March 19, 2025 12:12

carlos-lopez-tecnativa suggested changes Mar 19, 2025

View reviewed changes

chienandalu force-pushed the 17.0-MIG-mail_gateway branch 2 times, most recently from a5af4b4 to 4850108 Compare March 20, 2025 11:18

[MIG] mail_gateway: Migration to 17.0

56e5c2d

TT50728

chienandalu force-pushed the 17.0-MIG-mail_gateway branch from 4850108 to 56e5c2d Compare March 20, 2025 11:28

CarlosRoca13 approved these changes Jul 21, 2025

View reviewed changes

carlos-lopez-tecnativa approved these changes Jul 22, 2025

View reviewed changes

OCA-git-bot added the approved label Jul 22, 2025

OCA-git-bot added this to the 17.0 milestone Jul 22, 2025

OCA-git-bot added the bot is merging ⏳ label Jul 22, 2025

OCA-git-bot mentioned this pull request Jul 22, 2025

Migration to version 17.0 #1251

Open

40 tasks

OCA-git-bot merged commit 319405b into OCA:17.0 Jul 22, 2025
4 of 7 checks passed

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels Jul 22, 2025

pedrobaeza deleted the 17.0-MIG-mail_gateway branch July 22, 2025 13:50

pedrobaeza pushed a commit to Tecnativa/social that referenced this pull request Sep 22, 2025

[MIG] mail_gategay_telegram: Migration to v17

fcf61d2

WIP for test workflows of mail_gateway migration in OCA#1614

Uh oh!

[17.0] mail_gateway: Migration to 17.0 #1614

[17.0] mail_gateway: Migration to 17.0 #1614

Uh oh!

Conversation

chienandalu commented Mar 5, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

carlos-lopez-tecnativa left a comment

Choose a reason for hiding this comment

Uh oh!

carlos-lopez-tecnativa Mar 19, 2025

Choose a reason for hiding this comment

Uh oh!

chienandalu Mar 20, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

carlos-lopez-tecnativa Mar 19, 2025

Choose a reason for hiding this comment

Uh oh!

chienandalu Mar 20, 2025

Choose a reason for hiding this comment

Uh oh!

carlos-lopez-tecnativa Mar 20, 2025

Choose a reason for hiding this comment

Uh oh!

chienandalu Mar 20, 2025

Choose a reason for hiding this comment

Uh oh!

carlos-lopez-tecnativa Mar 20, 2025

Choose a reason for hiding this comment

Uh oh!

CarlosRoca13 commented Jul 21, 2025

Uh oh!

carlos-lopez-tecnativa left a comment

Choose a reason for hiding this comment

Uh oh!

pedrobaeza commented Jul 22, 2025

Uh oh!

OCA-git-bot commented Jul 22, 2025

Uh oh!

Uh oh!

OCA-git-bot commented Jul 22, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

10 participants

chienandalu commented Mar 5, 2025 •

edited

Loading