Skip to content

PM-6430 - campus leaderboad - #167

Merged
vas3a merged 1 commit into
developfrom
PM-6430_campus-leaderboard
Oct 5, 2026
Merged

vas3a merged 1 commit into
developfrom
PM-6430_campus-leaderboard

Conversation

@vas3a

@vas3a vas3a commented Oct 5, 2026

Copy link
Copy Markdown
Collaborator

This pull request improves how parent/subgroup relationships are handled in the campus-leaderboard.sql report by unifying the logic for reading group hierarchies. The main enhancement is the introduction of a new group_links common table expression (CTE) to ensure that group relationships are consistently captured from both the GroupMember and _ParentSubGroups tables, making the group tree traversal more robust.

Improvements to group hierarchy handling:

  • Added a new group_links CTE that merges parent/subgroup relationships from both groups."GroupMember" and groups."_ParentSubGroups" tables, ensuring no missing links due to incomplete data in either source.
  • Updated the group_tree and parent_groups CTEs to use group_links instead of directly querying GroupMember, simplifying and unifying the logic for traversing group hierarchies.

@vas3a
vas3a merged commit 27d607f into develop Oct 5, 2026
6 checks passed
@vas3a
vas3a deleted the PM-6430_campus-leaderboard branch October 5, 2026 11:35
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.

1 participant