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

[IMP] Improve Visitor integrations (Automation + Email/SMS Marketing) #40247

Open

Conversation

@std-odoo
Copy link
Contributor

std-odoo commented Nov 14, 2019

  • mass_mailing: allow users to select website.visitor as a target for a SMS/Email marketing
  • website: make the visitor model support the mailing flow

Task #2082388

@robodoo robodoo added the seen 🙂 label Nov 14, 2019
@C3POdoo C3POdoo added the RD label Nov 14, 2019
@robodoo robodoo added the CI 🤖 label Nov 14, 2019
@std-odoo std-odoo force-pushed the odoo-dev:master-website-improve-visitor-integration-2019-11-14-std branch from ea8d66d to cfd20bc Nov 21, 2019
@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Nov 21, 2019
@dbeguin

This comment has been minimized.

Copy link
Contributor

dbeguin commented Nov 21, 2019

- the field ``mobile`` and ``email`` need to be stored, because the method ``_get_seen_list`` use a SQL request
change that to
- Store mobile and email fields to make visitor model usable in the mass mailing process as _get_seen_list make a SQL request on those fields
I think this is more clear like this.

@std-odoo std-odoo force-pushed the odoo-dev:master-website-improve-visitor-integration-2019-11-14-std branch from cfd20bc to 64a09c7 Nov 21, 2019
@std-odoo

This comment has been minimized.

Copy link
Contributor Author

std-odoo commented Nov 21, 2019

- the fieldmobileandemailneed to be stored, because the method_get_seen_listuse a SQL request
change that to
- Store mobile and email fields to make visitor model usable in the mass mailing process as _get_seen_list make a SQL request on those fields
I think this is more clear like this.

@dbeguin done :)

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Nov 21, 2019
Copy link
Contributor

dbeguin left a comment

LGTM !

@std-odoo std-odoo force-pushed the odoo-dev:master-website-improve-visitor-integration-2019-11-14-std branch from 64a09c7 to bbb445c Dec 3, 2019
@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Dec 3, 2019
@std-odoo std-odoo force-pushed the odoo-dev:master-website-improve-visitor-integration-2019-11-14-std branch from bbb445c to 70922e9 Dec 9, 2019
@robodoo robodoo removed the CI 🤖 label Dec 9, 2019
@std-odoo std-odoo force-pushed the odoo-dev:master-website-improve-visitor-integration-2019-11-14-std branch from 70922e9 to 4d8b2dc Dec 9, 2019
@robodoo robodoo added the CI 🤖 label Dec 9, 2019
@std-odoo std-odoo force-pushed the odoo-dev:master-website-improve-visitor-integration-2019-11-14-std branch from 4d8b2dc to d0e68e1 Dec 10, 2019
@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Dec 10, 2019
@std-odoo std-odoo force-pushed the odoo-dev:master-website-improve-visitor-integration-2019-11-14-std branch from d0e68e1 to 3d1f2f3 Jan 3, 2020
@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Jan 3, 2020
@std-odoo std-odoo force-pushed the odoo-dev:master-website-improve-visitor-integration-2019-11-14-std branch from 3d1f2f3 to c8df65f Jan 6, 2020
@robodoo robodoo removed the CI 🤖 label Jan 6, 2020
@std-odoo std-odoo force-pushed the odoo-dev:master-website-improve-visitor-integration-2019-11-14-std branch from c8df65f to 227e16c Jan 6, 2020
@robodoo robodoo added the CI 🤖 label Jan 6, 2020
@std-odoo std-odoo force-pushed the odoo-dev:master-website-improve-visitor-integration-2019-11-14-std branch from 227e16c to cfcfa5d Jan 6, 2020
@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Jan 6, 2020
@std-odoo std-odoo force-pushed the odoo-dev:master-website-improve-visitor-integration-2019-11-14-std branch from cfcfa5d to 7253911 Jan 6, 2020
@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Jan 6, 2020
@std-odoo std-odoo force-pushed the odoo-dev:master-website-improve-visitor-integration-2019-11-14-std branch from 7253911 to a10bde1 Jan 6, 2020
@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Jan 6, 2020
@std-odoo std-odoo force-pushed the odoo-dev:master-website-improve-visitor-integration-2019-11-14-std branch from a10bde1 to f6c8f9f Jan 7, 2020
@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Jan 7, 2020
Copy link
Contributor

tde-banana-odoo left a comment

Riviou

addons/website/models/website_visitor.py Outdated Show resolved Hide resolved
addons/website/models/website_visitor.py Outdated Show resolved Hide resolved
addons/website_mail/__manifest__.py Outdated Show resolved Hide resolved
addons/website_mail/data/website_mail_data.xml Outdated Show resolved Hide resolved
std-odoo added a commit to odoo-dev/odoo that referenced this pull request Feb 12, 2020
Purpose
=======
Make the visitor model support the mailing flow, so we can chat
next to the visitor like we do for the partner or the lead.

Specifications
==============
Add the option "enable_sms" on the mobile field,
so we can click on it to open the SMS composer.

The logs when we send a Mail/SMS are now in the chatter of visitor

Task 2082388
PR odoo#40247
std-odoo added a commit to odoo-dev/odoo that referenced this pull request Feb 12, 2020
@std-odoo std-odoo force-pushed the odoo-dev:master-website-improve-visitor-integration-2019-11-14-std branch from f6c8f9f to f09b108 Feb 12, 2020
@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Feb 12, 2020
std-odoo added 3 commits Nov 14, 2019
Purpose
=======
Make the visitor model support the mailing flow, so we can chat
next to the visitor like we do for the partner or the lead.

Specifications
==============
Add the option "enable_sms" on the mobile field,
so we can click on it to open the SMS composer.

The logs when we send a Mail/SMS are now in the chatter of visitor

Task 2082388
PR #40247
- the field ``mobile`` and ``email`` need to be stored, because the method ``_get_seen_list`` use a SQL request
- add the model ``website.visitor`` in ``MASS_MAILING_BUSINESS_MODELS`` so it can be selected

Task #2082388
@std-odoo std-odoo force-pushed the odoo-dev:master-website-improve-visitor-integration-2019-11-14-std branch from f09b108 to c2641b7 Feb 14, 2020
@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Feb 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.