Skip to content

Change from valueCode to parentCode as keys#169

Merged
JohannesFinsveen merged 1 commit intofix/multi-groupingfrom
fix/multi-grouping-parent-as-key
Jun 20, 2025
Merged

Change from valueCode to parentCode as keys#169
JohannesFinsveen merged 1 commit intofix/multi-groupingfrom
fix/multi-grouping-parent-as-key

Conversation

@JohannesFinsveen
Copy link
Collaborator

Does children have parents or does parents have children :-)

@sonarqubecloud
Copy link

@likp
Copy link
Contributor

likp commented Jun 19, 2025

This PR make sens, I did not dare to change the code to much.

Since we are tidying up the code it what is the purpose of GetValueGroupRowsByParentCode It seams to just do a shallow copy of the same Dictonary that it get from GetValueGroupRowsByParentCodeFromBucket

@JohannesFinsveen
Copy link
Collaborator Author

Doesnt it split unlikely long lists of parentCodes(500, hardcoded :-) ) into chunks(buckets), loop the chunks and glue the result together... But could the dict be (string , List) ? Meeting tomorrow?

@likp
Copy link
Contributor

likp commented Jun 19, 2025

You are right I missed the call to CreateGroupCodeBuckets.

We have holiday tomorrow, Midsommar. But I think it looks good. I will approved it shortly.

Copy link
Contributor

@likp likp left a comment

Choose a reason for hiding this comment

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

LGTM

@JohannesFinsveen
Copy link
Collaborator Author

We have holiday tomorrow, Midsommar.

But St. Hans is on monday :-) Enjoy!

@JohannesFinsveen JohannesFinsveen merged commit 1b62bca into fix/multi-grouping Jun 20, 2025
3 checks passed
@JohannesFinsveen JohannesFinsveen deleted the fix/multi-grouping-parent-as-key branch July 3, 2025 07:44
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