Skip to content
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

[10.0][MIG] mass_mailing_unique #188

Merged
merged 5 commits into from
Aug 7, 2017
Merged

Conversation

pedrobaeza
Copy link
Member

Standard migration

cc @Tecnativa

Jairo Llopis and others added 4 commits August 4, 2017 20:01
This new module avoids duplicates in mass mailing lists.

Now you cannot have 2 lists with the same name, and you cannot have the same
email repeated in a list.

This will avoid sending the same mail to a contact repeated times, which will
disturb him and most probably cause you to be blocked as spam.
@pedrobaeza
Copy link
Member Author

pedrobaeza commented Aug 4, 2017

Relevant commit is 92ff260

@pedrobaeza
Copy link
Member Author

@chienandalu @cubells please review

@pedrobaeza pedrobaeza mentioned this pull request Aug 4, 2017
21 tasks
Copy link
Member

@yajo yajo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code & functional review

@yajo yajo added this to the 10.0 milestone Aug 7, 2017
Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Sponsor Member

@cubells cubells left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

LGTM

Code review and test.

@yajo yajo merged commit 7152fac into OCA:10.0 Aug 7, 2017
@yajo yajo deleted the 10.0-mass_mailing_unique branch August 7, 2017 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants