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

Saved Question: changing the breakout field (summarize) removes order-by (sort) #17975

Closed
ariya opened this issue Sep 20, 2021 · 0 comments · Fixed by #17982
Closed

Saved Question: changing the breakout field (summarize) removes order-by (sort) #17975

ariya opened this issue Sep 20, 2021 · 0 comments · Fixed by #17982
Assignees
Labels
.Frontend Priority:P3 Cosmetic bugs, minor bugs with a clear workaround Querying/Nested Queries Questions based on other saved questions Querying/Notebook Items specific to the Custom/Notebook query builder .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects
Milestone

Comments

@ariya
Copy link
Contributor

ariya commented Sep 20, 2021

To Reproduce

  1. Ask a question, native question.
  2. Choose Sample Dataset, type select * from orders.
  3. Save it as SQL Orders.
  4. Ask a question, Custom question.
  5. Choose Saved Questions, SQL Orders.
  6. Summarize: Count, by CREATED_AT: Year.
  7. Add sorting, choose CREATED_AT
  8. Now go back and change the summarization to CREATED_AT: Month

Actual

Sort is remove.

image

Expected

Sort is still there.

@ariya ariya added Type:Bug Product defects .Needs Triage labels Sep 20, 2021
@flamber flamber added Priority:P3 Cosmetic bugs, minor bugs with a clear workaround Querying/Notebook Items specific to the Custom/Notebook query builder Querying/Nested Queries Questions based on other saved questions .Frontend and removed .Needs Triage labels Sep 20, 2021
@ariya ariya self-assigned this Sep 20, 2021
ariya added a commit to ariya/metabase that referenced this issue Sep 20, 2021
ariya added a commit that referenced this issue Sep 20, 2021
@ariya ariya added the .Reproduced Issues reproduced in test (usually Cypress) label Sep 20, 2021
ariya added a commit that referenced this issue Sep 20, 2021
@ariya ariya linked a pull request Sep 21, 2021 that will close this issue
@flamber flamber added this to the 0.41 milestone Sep 23, 2021
This was referenced May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
.Frontend Priority:P3 Cosmetic bugs, minor bugs with a clear workaround Querying/Nested Queries Questions based on other saved questions Querying/Notebook Items specific to the Custom/Notebook query builder .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants