-
-
Notifications
You must be signed in to change notification settings - Fork 588
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
[9.0][MIG] mail_tracking_mass_mailing module #144
[9.0][MIG] mail_tracking_mass_mailing module #144
Conversation
f6dd469
to
3c75485
Compare
[ADD] mail_tracking_mass_mailing ============================== Mail tracking for mass mailing ============================== Links mail statistics objects with mail tracking objects. Installation ============ This addon will be automatically installed when 'mail_tracking' and 'mass_mailing' are both installed Usage ===== From mail statistic object, you can see: - Email tracking state - Email related tracking object - Email related tracking events From mass mailing contact, you can see: - Email score, in order to clean up your lists from bad score emails As a bonus feature, you have a new checkbox 'Avoid resend' in mass mailing, in order to not send twice the same email to the same recipient. This is very useful when you want to resend the mass mailing after changing selection recipients. Notice that recipient selection could be a domain over a model, so result ids could change over the time. With this flag you can send the same email several times but only once to each recipient.
3c75485
to
deb1bd5
Compare
Please review the change in the signature of the method |
This is still failing, can you check? |
837f7ef
to
32e2dd8
Compare
Rebased to test travis error. |
The error is still here. Please check. |
f0949a8
to
77f3566
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Working in production enviroment with this PR. All is working fine
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please squash last 2 commits and I'll merge
@cubells, please squash last 2 commits |
77f3566
to
c1d9562
Compare
@pedrobaeza |
cc @Tecnativa