Skip to content

add super admin section to admin sidebar#38

Merged
balebbae merged 2 commits intomainfrom
feat/super-admin-sidebar
Feb 27, 2026
Merged

add super admin section to admin sidebar#38
balebbae merged 2 commits intomainfrom
feat/super-admin-sidebar

Conversation

@jagadeep298218
Copy link
Collaborator

@jagadeep298218 jagadeep298218 commented Feb 26, 2026

Summary

Adds a dedicated "Super Admin" section to the admin sidebar that only renders for users with the super_admin role.

Also resolves a high-severity security vulnerability in minimatch that was blocking CI.

Changes

Feature: Super Admin Sidebar Section

  • Added superAdminNav items array with a Dashboard link (/superadmin)
  • Conditionally render new NavSection after the "Event" section when user?.role === 'super_admin'
  • Imported LayoutDashboard icon from lucide-react

Security Fix

Implementation Details

  • Follows existing NavSection component pattern
  • Uses existing role check pattern: user?.role === 'super_admin'
  • Super admin pages are already wrapped with <RequireSuperAdmin> in routes.tsx

Screenshot

image

Copy link
Collaborator

@balebbae balebbae left a comment

Choose a reason for hiding this comment

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

lgtm

@balebbae balebbae force-pushed the feat/super-admin-sidebar branch from bc59a8d to 927964c Compare February 27, 2026 00:10
@balebbae balebbae merged commit 6347ee3 into main Feb 27, 2026
4 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.

2 participants