Skip to content

Fix: Make university names case-insensitive in dashboard registration graph - #25

Merged
SayuruRehan merged 2 commits into
mainfrom
fix/university-names-case-insensitive
Oct 6, 2025
Merged

Fix: Make university names case-insensitive in dashboard registration graph#25
SayuruRehan merged 2 commits into
mainfrom
fix/university-names-case-insensitive

Conversation

@icy-r

@icy-r icy-r commented Oct 5, 2025

Copy link
Copy Markdown
Member

Changes Made

  • Modified src/components/dashboard/registrations.jsx to group universities by normalized (lowercase) name
  • Universities with the same name but different cases (e.g., "SLIIT" and "sliit") are now aggregated together
  • The first occurrence's casing is preserved for display purposes
  • Counts are summed up for all case variations of the same university name
  • Reverted pnpm-lock.yaml to its original state (unintended changes removed)

Fixes #23


Note: This PR replaces #24 which had commit history issues that could not be resolved without force push.

Copilot AI and others added 2 commits October 5, 2025 12:46
Co-authored-by: icy-r <87885588+icy-r@users.noreply.github.com>
Co-authored-by: icy-r <87885588+icy-r@users.noreply.github.com>
@vercel

vercel Bot commented Oct 5, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
bashaway-admin-portal Ready Ready Preview Comment Oct 5, 2025 0:47am

@SayuruRehan
SayuruRehan merged commit e2b2ec6 into main Oct 6, 2025
5 checks passed
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.

update dashboard team graph to consider university names as non case sensitive values

3 participants