Skip to content

Fix stacking bar chart#7516

Merged
yoshiokatsuneo merged 1 commit intogetredash:masterfrom
yoshiokatsuneo:fix/bar_chart_stacking
Aug 29, 2025
Merged

Fix stacking bar chart#7516
yoshiokatsuneo merged 1 commit intogetredash:masterfrom
yoshiokatsuneo:fix/bar_chart_stacking

Conversation

@yoshiokatsuneo
Copy link
Copy Markdown
Contributor

@yoshiokatsuneo yoshiokatsuneo commented Aug 29, 2025

What type of PR is this?

  • Bug Fix

Description

Fix the issue that stacking bar chart is not stacked.
This issue was introduced by #7514 , and this PR fixes the issue.

How is this tested?

  • Manually

I tested by running following query on Query Result datasource. and make stacking bar chart visualization.

select 1 as id, 10 as val, 'a' as grp
union all
select 2 as id, 30 as val, 'a' as grp
union all
select 3 as id, 20 as val, 'a' as grp
union all
select 1 as id, 11 as val, 'b' as grp
union all
select 2 as id, 31 as val, 'b' as grp
union all
select 3 as id, 21 as val, 'b' as grp

Setting:
image

Before applying this PR:
image

After applying this PR:
image

@yoshiokatsuneo yoshiokatsuneo force-pushed the fix/bar_chart_stacking branch from c37961b to 8837689 Compare August 29, 2025 14:09
Copy link
Copy Markdown
Collaborator

@eradman eradman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This fix works! Thanks for catching this @yoshiokatsuneo

@yoshiokatsuneo yoshiokatsuneo merged commit e586ab7 into getredash:master Aug 29, 2025
11 checks passed
@yoshiokatsuneo
Copy link
Copy Markdown
Contributor Author

@eradman Thank you for your quick review!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants