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

Pie chart title doesn't show #796

Closed
kewisch opened this issue Feb 1, 2019 · 4 comments
Closed

Pie chart title doesn't show #796

kewisch opened this issue Feb 1, 2019 · 4 comments
Assignees
@washort
Labels
merge upstream verified fixed
Milestone
20

Comments

@kewisch
Copy link

kewisch commented Feb 1, 2019

Issue Summary

The pie chart is incorrectly titled.

Steps to Reproduce

  1. Create a pie chart and set X and Y columns
  2. Go to the series tab and set a label for the series.

Results:

  • The pie chart is still titled by the name of the column

Expected:

  • The pie chart should be titled using the label for the series

Technical details:

  • Redash Version: 6.0.0 (dev) / STMO
@kewisch
Copy link
Author

kewisch commented Feb 1, 2019

image

@rafrombrc rafrombrc added this to the 20 milestone Feb 20, 2019
@washort washort added the in review upstream PR in review in getredash/redash label Feb 23, 2019
@washort
Copy link

washort commented Feb 23, 2019

getredash#3484

@washort washort added in review and removed ready labels Feb 23, 2019
@jezdez
Copy link

jezdez commented Feb 25, 2019

This has been merged upstream!

@jezdez jezdez closed this as completed Feb 25, 2019
@jezdez jezdez added merge upstream and removed in review merge upstream in review upstream PR in review in getredash/redash labels Feb 25, 2019
@madalincm
Copy link

Verified in FF66(Win7) in STMO-RC
Issue is no longer reproducing:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@washort washort

Labels
merge upstream verified fixed
Projects
None yet
Milestone
20
Development

No branches or pull requests
5 participants
@jezdez @rafrombrc @kewisch @washort @madalincm