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] mass_mailing: create the pivot view and update the values of graph view. #31307

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@rja-odoo
Copy link
Contributor

rja-odoo commented Feb 21, 2019

Task: https://www.odoo.com/web#id=1936692&action=333&active_id=131&model=project.task&view_type=form&menu_id=4720
Pad: https://pad.odoo.com/p/r.aa1a293f05c1870b41869ade471f759d

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@robodoo robodoo added the seen 🙂 label Feb 21, 2019

@C3POdoo C3POdoo added the RD label Feb 21, 2019

@rja-odoo rja-odoo force-pushed the odoo-dev:master-pivot-graph-view-rja branch 2 times, most recently Feb 21, 2019

[IMP] mass_mailing: create the pivot view and update the values of gr…
…aph view.

- The purpose of this commit is, graph views does not contain the right measures and pivot view is not there.
- Earlier, the following fields are non-storable (sent, received, opened, replied, clicks, bounced) and those fields can not be shown in graph because graph view perform aggregations on database values, they do not work with non-storable computed fields. So, to overcome from this, we make the fields storable.

Task Id #1936692

@rja-odoo rja-odoo force-pushed the odoo-dev:master-pivot-graph-view-rja branch to 6145f91 Feb 21, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.