New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] mail: Failed mail notification wrong title #30134

Closed
wants to merge 1 commit into
base: 12.0
from

Conversation

Projects
None yet
4 participants
@jpp-odoo
Copy link
Contributor

jpp-odoo commented Jan 11, 2019

opw-1921686

The failed mail notification group all the records of the same
model that failed to send an email.

Before this commit, the failed mail notification displayed as
title the name of the last record that failed to send an email.

Now, the notification displays as title the model name.

@robodoo robodoo added the seen 🙂 label Jan 11, 2019

@jpp-odoo

This comment has been minimized.

Copy link
Contributor

jpp-odoo commented Jan 11, 2019

@nim-odoo please review

[FIX] mail: Failed mail notification wrong title
opw-1921686

The failed mail notification group all the records of the same
model that failed to send an email.

Before this commit, the failed mail notification displayed as
title the name of the last record that failed to send an email.

Now, the notification displays as title the model name.

@jpp-odoo jpp-odoo force-pushed the odoo-dev:12.0-opw-1921686-fail_mail_notification-jpp branch from 058e7dc to 660be7d Jan 11, 2019

@jpp-odoo jpp-odoo changed the title [FIX] mail: Failed mail wrong notification title [FIX] mail: Failed mail notification wrong title Jan 11, 2019

@C3POdoo C3POdoo added the OE label Jan 11, 2019

@robodoo robodoo added the CI 🤖 label Jan 11, 2019

@nim-odoo
Copy link
Contributor

nim-odoo left a comment

robodoo r+

@robodoo robodoo added the r+ 👌 label Jan 11, 2019

robodoo pushed a commit that referenced this pull request Jan 11, 2019

[FIX] mail: Failed mail notification wrong title
opw-1921686

The failed mail notification group all the records of the same
model that failed to send an email.

Before this commit, the failed mail notification displayed as
title the name of the last record that failed to send an email.

Now, the notification displays as title the model name.

closes #30134
@robodoo

This comment has been minimized.

Copy link
Contributor

robodoo commented Jan 11, 2019

Merged, thanks!

@robodoo robodoo closed this Jan 11, 2019

@jpp-odoo jpp-odoo deleted the odoo-dev:12.0-opw-1921686-fail_mail_notification-jpp branch Jan 11, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment