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

[11.0][MIG] mail_tracking_mass_mailing: Migration to 11.0 #293

Merged
merged 9 commits into from
Aug 10, 2018

Conversation

pedrobaeza
Copy link
Member

  • Standard procedure
  • Adapt tests
  • New README by fragments

@Tecnativa

antespi and others added 9 commits August 9, 2018 19:23
[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.
Testing this addon failed if i.e. `mass_mailing_list_dynamic` was installed before.

Moving tests to post-install mode fixes integration tests.
* Standard procedure
* Adapt tests
* New README by fragments
@pedrobaeza pedrobaeza added this to the 11.0 milestone Aug 9, 2018
@rafaelbn
Copy link
Member

rafaelbn commented Aug 9, 2018

Hi, @pedrobaeza I cannot test it because runbot is nor available. Even rebuilding is not available 🤔

Copy link
Member

@rafaelbn rafaelbn left a comment

Choose a reason for hiding this comment

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

A leart tested in runbot 👍

I would like to think that in 3 years with this modules I've contribute for it....

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.

Looks good 👍

@pedrobaeza pedrobaeza mentioned this pull request Aug 10, 2018
26 tasks
@pedrobaeza pedrobaeza merged commit 823a613 into OCA:11.0 Aug 10, 2018
@pedrobaeza pedrobaeza deleted the 11.0-mail_tracking_mass_mailing branch August 10, 2018 05:15
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

8 participants