Skip to content

[IMP] mail: no email to partner followers of non-email channels - #33727

Closed
alexkuhn wants to merge 1 commit into
odoo:masterfrom
odoo-dev:master-mail-channel-send-email-to-partner-only-if-mass-mailing-set-aku
Closed

[IMP] mail: no email to partner followers of non-email channels#33727
alexkuhn wants to merge 1 commit into
odoo:masterfrom
odoo-dev:master-mail-channel-send-email-to-partner-only-if-mass-mailing-set-aku

Conversation

@alexkuhn

Copy link
Copy Markdown
Contributor

Do not send email to partner followers of non-mass-mailing channels.

When a partner follows a channel without "Send messages by email",
we expect the follower to not receive any email whatsoever.

This behaviour is required to implement mobile channel notifications
for some users without receiving them by email. Basically, users can
follow a channel to enable push notifications.

Note that a mention of the user in a non-mass-mailing channel still
sends an email to this user, in case the notification preferences are
set to "Handle by emails".

Do not send email to partner followers of non-mass-mailing channels.

When a partner follows a channel without "Send messages by email",
we expect the follower to not receive any email whatsoever.

This behaviour is required to implement mobile channel notifications
for some users without receiving them by email. Basically, users can
follow a channel to enable push notifications.

Note that a mention of the user in a non-mass-mailing channel still
sends an email to this user, in case the notification preferences are
set to "Handle by emails".
@C3POdoo C3POdoo added the RD research & development, internal work label May 28, 2019
@alexkuhn alexkuhn closed this Dec 3, 2019
@alexkuhn
alexkuhn deleted the master-mail-channel-send-email-to-partner-only-if-mass-mailing-set-aku branch December 3, 2019 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

RD research & development, internal work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants