[data grid] Nested grouping and sort by aggregation does not sort the second level

[mdoverhag]

[data grid] Nested grouping and sort by aggregation does not sort the second level

[mdoverhag]

Hi, first bug report here. Thanks for a cool product.

We just recently started using X Data Grid Premium and I was going to try and update from version 6.19.8 to 7.1.0.

In 6.19.8 I seem to be able to get above to work as expected. But the sorting does no longer seem to work correctly in 7.0.0 or later.

[michelengelen]

Hey @mdoverhag and thanks for raising an issue here.
I am glad you like the product! ✨

The use case you have there is pretty advanced.
I just tried it and you are right. The second sorting does not work.

This clearly is a regression, so I will add this to our board

v6: https://stackblitz.com/edit/github-zzppol-p1ndqj

Screen.Recording.2024-04-05.at.10.55.09.mov

v7: https://stackblitz.com/edit/github-zzppol-vhewiz

Screen.Recording.2024-04-05.at.10.52.58.mov

[mdoverhag]

Thanks for the prompt reply @michelengelen 👍 I'll keep a look out here then!

[cherniavskii]

Hi @mdoverhag
I checked both v6 and v7 (added links to demos in #12684 (comment)) and it seems to me that the behavior in v7 is correct.
In v6, sorting on the grouping column was only applied to the first level of the rows:

v6	v7

This was fixed #9679

We're grouping by Year-Month (have a separate string column for that). Then we group by sales person. We'd like to order the expanded Year-Month by the sales person with the most sales at the top.

Did you consider using multiple grouping columns for your use case?
Here's a working demo with Data Grid v7 https://stackblitz.com/edit/github-zzppol-u2w6mf

[mdoverhag]

Hi @cherniavskii,

Thank you for making the videos comparing v6 and v7. I get it, considering we're sorting on Grouping, the second level is also technically part of that column and it should be sorted alphabetically. 👍

Thanks for the suggestion on multiple grouping. We'll give that ago!

[github-actions]

⚠️ This issue has been closed. If you have a similar problem but not exactly the same, please open a new issue.
Now, if you have additional information related to this issue or things that could help future readers, feel free to leave a comment.

@mdoverhag: How did we do? Your experience with our support team matters to us. If you have a moment, please share your thoughts in this short Support Satisfaction survey.