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

Not possible to drill-through on combined cards with different amount of series #13457

Closed
flamber opened this issue Oct 13, 2020 · 0 comments · Fixed by #16608
Closed

Not possible to drill-through on combined cards with different amount of series #13457

flamber opened this issue Oct 13, 2020 · 0 comments · Fixed by #16608
Assignees
Labels
Priority:P2 Average run of the mill bug Reporting/Dashboards .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects Visualization/
Milestone

Comments

@flamber
Copy link
Contributor

flamber commented Oct 13, 2020

Describe the bug
When combining multi-series on dashboard, then it's only possible to drill-through on the same amount of series that is in the original card as the combined card has. So if original card contains 2 series, and combining with added card with 3 series, then it's not possible to drill-through on the added card's 3rd (last) series.

To Reproduce

  1. Simple question > Sample Dataset > Orders > Summarize Count grouped by CreatedAt:Year - save as Q1
    image
  2. Simple question > Sample Dataset > Orders > Summarize "Average of Discount" and "Average of Quantity" grouped by CreatedAt:Year - save as Q2
    image
  3. Add Q1 to dashboard, how the card and click "Add series" and select Q2, and save dashboard:
    image
  4. It's possible to click Q1's line points (blue), and Q2's first series "Average of Discount" (green), but it's not possible to click Q2's "Average of Quantity" (orange) and browser console errors with following:
Firefox: Uncaught TypeError: m is null
Chrome: Uncaught TypeError: Cannot read property '0' of null
    at t.default (app-main.bundle.js?f740cf3b1da47a41d2da:5)
    at app-main.bundle.js?f740cf3b1da47a41d2da:5
    at Array.map (<anonymous>)
    at e.value (app-main.bundle.js?f740cf3b1da47a41d2da:5)
    at t.value (app-main.bundle.js?f740cf3b1da47a41d2da:5)
    at n.handleVisualizationClick (app-main.bundle.js?f740cf3b1da47a41d2da:5)
    at SVGCircleElement.t (app-main.bundle.js?f740cf3b1da47a41d2da:5)
    at SVGCircleElement.__onclick (vendor.bundle.js?f740cf3b1da47a41d2da:1)
    at x (app-main.bundle.js?f740cf3b1da47a41d2da:5)
    at SVGPathElement.<anonymous> (app-main.bundle.js?f740cf3b1da47a41d2da:5)

Information about your Metabase Installation:
Metabase 0.35.0, 0.35.4, 0.36.7, 0.37.0-rc2 - this used to work on 0.34.3

@flamber flamber added Type:Bug Product defects Priority:P2 Average run of the mill bug Visualization/ Reporting/Dashboards labels Oct 13, 2020
@nemanjaglumac nemanjaglumac added this to Backlog in Cypress Testing Dec 4, 2020
nemanjaglumac added a commit that referenced this issue Dec 7, 2020
nemanjaglumac added a commit that referenced this issue Dec 8, 2020
…eries (#13987)

* Start each test in isolation

* Add repro for #13457
@nemanjaglumac nemanjaglumac added the .Reproduced Issues reproduced in test (usually Cypress) label Dec 8, 2020
@daltojohnso daltojohnso self-assigned this Jun 15, 2021
@flamber flamber added this to the 0.40 milestone Jun 16, 2021
This was referenced Apr 18, 2023
This was referenced Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:P2 Average run of the mill bug Reporting/Dashboards .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects Visualization/
Projects
Development

Successfully merging a pull request may close this issue.

3 participants