-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
Fix pie rows settings include filtered out values #47879
Fix pie rows settings include filtered out values #47879
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The problem with this approach is that when you edit pie.rows
and save the question after rows have been removed from the query result, when those rows reappear they will no longer have their settings. This is why we kept track of the removed rows and marked them with hidden: true
instead of removing them from the setting. See the example below
Screen.Recording.2024-09-11.at.5.00.13.PM.mov
ee09f58
to
fb8dad3
Compare
@EmmadUsmani thanks, I fixed that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
Closes #47867
Description
After editing pie rows settings, if dataset returned fewer rows we still showed previously existing rows in the settings sidebar. This PR filters them out.
How to verify
Checklist