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.

Sign up for GitHub

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

Jump to bottom

Trend visualization requires time column to be the first otherwise it shows blank visualization #13710

Closed
flamber opened this issue Nov 6, 2020 · 2 comments
Closed

Trend visualization requires time column to be the first otherwise it shows blank visualization #13710

flamber opened this issue Nov 6, 2020 · 2 comments
Assignees
@alxnddr
Labels
.Frontend Priority:P2 Average run of the mill bug .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects Visualization/Scalars Numbers, progress bars, gauges
Projects
Cypress Testing

Comments

@flamber
Copy link
Contributor

flamber commented Nov 6, 2020 •
edited
Loading

Describe the bug
Trend visualization requires time column to be the first otherwise it shows blank visualization.

To Reproduce

  1. Custom question > Sample Dataset > Orders
  2. Summarize - do not select metric - group by Quantity:unbinned and CreatedAt:Month
    image
  3. Set visualization to Trend - just shows blank visualization
    image
  4. Change the summarize grouping, so CreatedAt:Month is the first with Quantity:unbinned as second, then it works:
    image

Information about your Metabase Installation:
Tested 0.35 thru 0.37.0.2

⬇️ Please click the 👍 reaction instead of leaving a +1 or update? comment
@flamber flamber added Type:Bug Product defects Priority:P2 Average run of the mill bug Visualization/Scalars Numbers, progress bars, gauges labels Nov 6, 2020
@nemanjaglumac nemanjaglumac added this to Backlog in Cypress Testing Nov 11, 2020
nemanjaglumac added a commit that referenced this issue Nov 11, 2020
@nemanjaglumac
Add repro for #13710 [ci skip]
2b7e611
@nemanjaglumac nemanjaglumac mentioned this issue Nov 11, 2020
Merged
nemanjaglumac added a commit that referenced this issue Nov 12, 2020
@nemanjaglumac
#13710 repro: Trend visualization blank screen [ci skip] (#13753)
7a1f605
@nemanjaglumac nemanjaglumac added the .Reproduced Issues reproduced in test (usually Cypress) label Nov 12, 2020
@flamber flamber mentioned this issue Jan 5, 2021
Closed
@alxnddr
Copy link
Member

alxnddr commented Jun 16, 2021

@metabase/backend-developers it looks like a back end fix is requires for this bug because the insights prop does not have correct data:
Screen Shot 2021-06-16 at 22 27 52
last-change and previous-value are missing

@alxnddr alxnddr mentioned this issue Jun 16, 2021
Merged
@alxnddr alxnddr self-assigned this Jun 18, 2021
This was referenced Jun 18, 2021
Open
Merged
@alxnddr alxnddr removed the .Backend label Jun 18, 2021
@alxnddr
Copy link
Member

alxnddr commented Jun 18, 2021

Closed by #16638

Created a separate issue for missing BE data: #16683

@alxnddr alxnddr closed this as completed Jun 18, 2021
@deploysentinel deploysentinel bot mentioned this issue Mar 7, 2023
Merged
@deploysentinel deploysentinel bot mentioned this issue Nov 6, 2023
Merged
@deploysentinel deploysentinel bot mentioned this issue Dec 7, 2023
Closed
This was referenced Jan 11, 2024
Merged
Merged
Merged
Merged
Merged
Closed
Merged
Merged
This was referenced May 8, 2024
Closed
Merged
Merged
Merged
Merged
Merged
Merged
Closed
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alxnddr alxnddr

Labels
.Frontend Priority:P2 Average run of the mill bug .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects Visualization/Scalars Numbers, progress bars, gauges
Projects
Cypress Testing
Backlog
Milestone
No milestone
Development

No branches or pull requests
3 participants
@flamber @alxnddr @nemanjaglumac