[MIG] mail_tracking: Migration to 16.0 #1029

hvgollar · 2022-12-19T08:47:45Z

No description provided.

* [ADD] mail_tracking addon * Add description icon * Fixes remarked * Fix Travis error * Remarks fixed

…ent events (OCA#82) [IMP] mail_tracking: Speed installation time, discard concurrent events and other fixes

* Improve tests * Show trackings even if partner removed * Disable CSRF protection to webhooks controllers

…s and contacts to bounce (OCA#133)

…oved (OCA#132)

As regular users can't access this object.

* [FIX] Only one data-odoo-tracking-email tag in each email * [FIX] Get status even in multicompany instances

On our server, queries based on "mail_tracking_event"."tracking_email_id" improved from 501,924 ms to 1,840 ms queries based on "mail_tracking_email"."mail_message_id" improved from 167,436 ms to 3,223 ms The last ones are run several times when a thread has many messages

Currently translated at 100.0% (82 of 82 strings) Translation: social-11.0/social-11.0-mail_tracking Translate-URL: https://translation.odoo-community.org/projects/social-11-0/social-11-0-mail_tracking/fr/

Closes OCA#303

For giving more priority to other buttons like the invoices one.

Currently translated at 96.3% (79 of 82 strings) Translation: social-11.0/social-11.0-mail_tracking Translate-URL: https://translation.odoo-community.org/projects/social-11-0/social-11-0-mail_tracking/ca/

Or infinite recursions will happen on other `write` overwrites, like the one that happens on `mass_mailing_partner`.

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: social-15.0/social-15.0-mail_tracking Translate-URL: https://translation.odoo-community.org/projects/social-15-0/social-15-0-mail_tracking/

maq-adhoc

When the mails fail an error appears when rendering the chatter.

Uncaught Promise > Invalid props for component 'https://github.com/messaging MessageFailedBox': unknown key 'record', 'chatterLocalId' is missing (should be a string)

nicomacr · 2022-12-28T12:31:12Z

@hvgollar the error is still present in the module as the last comment said.

hvgollar · 2022-12-28T12:34:09Z

@hvgollar the error is still present in the module as the last comment said.

Could help me in reproducing the issue.

nicolascol · 2022-12-29T17:49:15Z

@hvgollar the error is still present in the module as the last comment said.

Could help me in reproducing the issue.

hi, the error is generated when entering the discuss app
@hvgollar

rconjour · 2023-01-04T12:49:38Z

@hvgollar can you please squash all your commits?

hvgollar · 2023-01-04T12:57:28Z

@hvgollar can you please squash all your commits?

@rconjour Yes, its done

nicomacr · 2023-01-25T22:16:08Z

@hvgollar I faced this bug using the module Promesa desconocida > "Directive t-foreach should always be used with a t-key!" (expression: t-foreach="messagefailed.failed_recipients" t-as="recipient") Is happened when creating an activity.
Could you check the JS part?

chienandalu

Hi @hvgollar

Consider this v15 PRs that bring up some assorted fixes:

chienandalu

You should respect attribution of the changes you're including (cherry-picking or rebasing) not to include those in the migration commit.

alan196 · 2023-02-22T13:32:11Z

I tested migration to V16 with a database with this module installed, It's failing to open views with failed messages.

It raises the following error in JS console.


Missing dependencies:     (6) ['mail.Manager.Notification', 'mail.model.AbstractMessage', 'mail.model.Message', 'mail.Discuss', 'mail.Manager', 'mail.model.Mailbox']0: "mail.Manager.Notification"1: "mail.model.AbstractMessage"2: "mail.model.Message"3: "mail.Discuss"4: "mail.Manager"5: "mail.model.Mailbox"length: 6[[Prototype]]: Array(0)
boot.js:347 Non loaded modules:       ['mail_tracking.FailedMessageDiscuss']0: "mail_tracking.FailedMessageDiscuss"length: 1[[Prototype]]: Array(0)
boot.js:355 Debug:                    {mail_tracking.FailedMessageDiscuss: {…}}mail_tracking.FailedMessageDiscuss: {dependencies: Array(8), dependents: Array(0), name: 'mail_tracking.FailedMessageDiscuss', missing: Array(6)}[[Prototype]]: Object
error_service.js:70 OwlError: Got duplicate key in t-foreach: false
    at MessagingClass.template (eval at compile (owl.js:5470:1), <anonymous>:232:51) (/web/static/lib/owl/owl.js:5470)
    at node.renderFn (owl.js:2633:1) (/web/static/lib/owl/owl.js:2633)
    at node.renderFn (owl.js:2642:1) (/web/static/lib/owl/owl.js:2642)
    at Fiber._render (owl.js:1621:1) (/web/static/lib/owl/owl.js:1621)
    at Fiber.render (owl.js:1613:1) (/web/static/lib/owl/owl.js:1613)
    at ComponentNode.initiateRender (owl.js:2352:1) (/web/static/lib/owl/owl.js:2352)

pedrobaeza · 2023-03-14T07:31:05Z

Superseded by #1091, as this one is not fully working.

antespi and others added 30 commits December 12, 2022 15:42

[8.0][ADD] mail_tracking addon (OCA#67)

9a41054

* [ADD] mail_tracking addon * Add description icon * Fixes remarked * Fix Travis error * Remarks fixed

[8.0][IMP][mail_tracking] Speed installation time and discard concurr…

e296356

…ent events (OCA#82) [IMP] mail_tracking: Speed installation time, discard concurrent events and other fixes

[FIX] Update email_score when email changes

5d44882

[MIG] mail_tracking

60fc01a

* Improve tests * Show trackings even if partner removed * Disable CSRF protection to webhooks controllers

[FIX] mail_tracking: Typo

ca0530e

[MIG] Make modules uninstallable

bc2f65a

[MIG] mail_tracking: Migrated to 10.0

33d98df

[IMP] mail_tracking performance and bounce process (OCA#103)

704bf9d

[IMP] mail_tracking: Allow to define a custom mail_tracking base url

4e2221b

Add access to portal/public user to his own email trackings

67f8ce0

[8.0][FIX][mail_tracking] Use event recipient address to find partner…

1a3a9f1

…s and contacts to bounce (OCA#133)

[9.0][FIX][mail_tracking] Control exception when mail_message was rem…

fd74467

…oved (OCA#132)

[FIX] Sender label is repeated in search fields

5f3b14c

[FIX][9.0] partner buttons view on enterprise

cde9e32

[FIX] mail_tracking: Obtain outgoing server with sudo

42113a0

As regular users can't access this object.

[IMP] Shut mock exceptions up

9b08019

[9.0][IMP] mail_tracking: Improve view with error details

1986189

[FIX] mail_tracking: bugfixes (OCA#199)

df416c6

* [FIX] Only one data-odoo-tracking-email tag in each email * [FIX] Get status even in multicompany instances

[FIX] mail_tracking js could break other modules using chat_manager

58d70ec

[MIG] mail_tracking: Migration to 11.0

1acc609

Translated using Weblate (French) [ci skip]

60476cc

Currently translated at 100.0% (82 of 82 strings) Translation: social-11.0/social-11.0-mail_tracking Translate-URL: https://translation.odoo-community.org/projects/social-11-0/social-11-0-mail_tracking/fr/

[IMP] mail_tracking: email score performance (OCA#299)

0ae63a2

Fix README.rst images not visible in GitHub and Odoo

8fa6cea

Closes OCA#303

[IMP] mail_tracking: Put button box at the end of the view

71faf76

For giving more priority to other buttons like the invoices one.

Translated using Weblate (Catalan)

fe75421

Currently translated at 96.3% (79 of 82 strings) Translation: social-11.0/social-11.0-mail_tracking Translate-URL: https://translation.odoo-community.org/projects/social-11-0/social-11-0-mail_tracking/ca/

[MIG] mail_tracking: Migration to 12.0

d73bc59

[FIX] mail_tracking: Don't call write inside other write

bfbda1f

Or infinite recursions will happen on other `write` overwrites, like the one that happens on `mass_mailing_partner`.

[IMP] mail_tracking: Cc

ad99ca8

[MIG] mail_tracking: Adapt Cc to 12.0

2794772

oca-ci and others added 8 commits December 12, 2022 15:42

[UPD] Update mail_tracking.pot

9f9a6a4

[UPD] README.rst

20a6876

mail_tracking 15.0.1.0.1

4e44eaa

Fixed Deprecated Warning

581aa34

mail_tracking 15.0.1.0.2

bdd67ab

[UPD] Update mail_tracking.pot

8e9a813

Update translation files

0a9aa4c

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: social-15.0/social-15.0-mail_tracking Translate-URL: https://translation.odoo-community.org/projects/social-15-0/social-15-0-mail_tracking/

[IMP] mail_tracking: pre-commit stuff

09ccc66

hvgollar mentioned this pull request Dec 19, 2022

Migration to version 15.0 #789

Open

36 tasks

maq-adhoc suggested changes Dec 22, 2022

View reviewed changes

hvgollar force-pushed the 16.0-mig-mail_tracking branch from 136d121 to 5123c4d Compare January 4, 2023 12:53

chienandalu reviewed Jan 31, 2023

View reviewed changes

rven force-pushed the 16.0-mig-mail_tracking branch from 6217a4a to dc449b1 Compare February 7, 2023 14:33

[MIG] mail_tracking: Migration to 16.0

b6ab7ef

rven force-pushed the 16.0-mig-mail_tracking branch from dc449b1 to b6ab7ef Compare February 7, 2023 14:35

chienandalu suggested changes Feb 7, 2023

View reviewed changes

alan196 mentioned this pull request Feb 21, 2023

[16.0][MIG] Migration of module mail_tracking_mailgun #1073

Closed

9 tasks

maq-adhoc mentioned this pull request Feb 24, 2023

16.0 mig mail tracking #1080

Closed

maq-adhoc mentioned this pull request Mar 7, 2023

[16.0][MIG] mail_tracking: Migration to 16.0 #1091

Closed

This was referenced Mar 9, 2023

16.0 mig mail tracking mass mailing nguyenhk99/social#1

Open

[16.0][MIG] mail_tracking_mass_mailing: Migration to 16.0 #1099

Closed

pedrobaeza closed this Mar 14, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[MIG] mail_tracking: Migration to 16.0 #1029

[MIG] mail_tracking: Migration to 16.0 #1029

hvgollar commented Dec 19, 2022

maq-adhoc left a comment

nicomacr commented Dec 28, 2022

hvgollar commented Dec 28, 2022

nicolascol commented Dec 29, 2022 •

edited

Loading

rconjour commented Jan 4, 2023

hvgollar commented Jan 4, 2023

nicomacr commented Jan 25, 2023

chienandalu left a comment •

edited

Loading

chienandalu left a comment

alan196 commented Feb 22, 2023

pedrobaeza commented Mar 14, 2023

[MIG] mail_tracking: Migration to 16.0 #1029

[MIG] mail_tracking: Migration to 16.0 #1029

Conversation

hvgollar commented Dec 19, 2022

maq-adhoc left a comment

Choose a reason for hiding this comment

nicomacr commented Dec 28, 2022

hvgollar commented Dec 28, 2022

nicolascol commented Dec 29, 2022 • edited Loading

rconjour commented Jan 4, 2023

hvgollar commented Jan 4, 2023

nicomacr commented Jan 25, 2023

chienandalu left a comment • edited Loading

Choose a reason for hiding this comment

chienandalu left a comment

Choose a reason for hiding this comment

alan196 commented Feb 22, 2023

pedrobaeza commented Mar 14, 2023

nicolascol commented Dec 29, 2022 •

edited

Loading

chienandalu left a comment •

edited

Loading