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] web: improved graph view for the group by more than two fields #29673

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
4 participants
@jpr-odoo
Copy link
Contributor

jpr-odoo commented Dec 20, 2018

Description of the issue/feature this PR addresses:
Task ID: https://www.odoo.com/web?#id=1917948&action=333&active_id=131&model=project.task&view_type=form&menu_id=4720

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 Dec 20, 2018

@C3POdoo C3POdoo added the RD label Dec 20, 2018

@robodoo robodoo added the CI 🤖 label Dec 20, 2018

@robodoo robodoo removed the CI 🤖 label Jan 1, 2019

@msh-odoo msh-odoo force-pushed the odoo-dev:master-graph-view-grouping-more-fields-jpr branch Jan 1, 2019

@robodoo robodoo added the CI 🤖 label Jan 3, 2019

@jpr-odoo jpr-odoo force-pushed the odoo-dev:master-graph-view-grouping-more-fields-jpr branch Jan 3, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Jan 3, 2019

@Polymorphe57 Polymorphe57 force-pushed the odoo-dev:master-graph-view-grouping-more-fields-jpr branch Feb 19, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Feb 19, 2019

[IMP] web: group by more than two fields in graph view
Before this commit: select more than two groupbys
in a graph view would have no impact at all if mode
'bar' or 'line' is selected.

This commit allows to correctly handle more than
two groupbys in all modes.

Task ID: 1917948

@Polymorphe57 Polymorphe57 force-pushed the odoo-dev:master-graph-view-grouping-more-fields-jpr branch to 02cfe7e Feb 20, 2019

@robodoo robodoo removed the CI 🤖 label Feb 20, 2019

@aab-odoo

This comment has been minimized.

Copy link
Contributor

aab-odoo commented Feb 20, 2019

robodoo r+

robodoo pushed a commit that referenced this pull request Feb 20, 2019

[IMP] web: group by more than two fields in graph view
Before this commit: select more than two groupbys
in a graph view would have no impact at all if mode
'bar' or 'line' is selected.

This commit allows to correctly handle more than
two groupbys in all modes.

Task ID: 1917948

closes #29673
@robodoo

This comment has been minimized.

Copy link
Contributor

robodoo commented Feb 20, 2019

Merged, thanks!

@robodoo robodoo closed this Feb 20, 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.