[FIX] mail_post_defer: notify from non-thread models and to author

OCA / social

Addons concerning odoo's social ERP features and messaging in general

https://odoo-community.org/project/99

GNU Affero General Public License v3.0

157 stars 600 forks source link

[FIX] mail_post_defer: notify from non-thread models and to author #1380

Closed yajo closed 3 months ago

yajo commented 3 months ago

In some places of Odoo, there are direct calls to self.env["mail.thread"].message_notify() where the target model isn't extending mail.thread. These cases were failing, due to the grouping done by mail.message.schedule.

In some of those cases, mails were sent with the special mail_notify_author=True context, which got lost after the deferring step. In those cases, if the author's message was expected to arrive at the author's inbox, it wouldn't happen anymore.

@moduon MT-6337

OCA-git-bot commented 3 months ago

Hi @Yajo, some modules you are maintaining are being modified, check this out!

yajo commented 3 months ago

Fasttracking this important fix.

/ocabot merge patch

OCA-git-bot commented 3 months ago

What a great day to merge this nice PR. Let's do it! Prepared branch 16.0-ocabot-merge-pr-1380-by-yajo-bump-patch, awaiting test results.

OCA-git-bot commented 3 months ago

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