-
Notifications
You must be signed in to change notification settings - Fork 19
feat(admin-ui): unused components cleanup and migration to single navigation wrapper #2513
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…i-issue-2361-auth-server
…i-issue-2361-auth-server
…i-issue-2361-saml-1
…i-issue-2361-saml-1
…i-issue-2361-saml-1
|
Important Review skippedMore than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review. 38 files out of 145 files are above the max files limit of 100. Please upgrade to Pro plan to get higher limits. You can disable this status message by setting the ✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
Signed-off-by: Faisal Siddique <71010439+faisalsiddique4400@users.noreply.github.com>
Signed-off-by: Faisal Siddique <71010439+faisalsiddique4400@users.noreply.github.com>
duttarnab
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- In user claims on changing the status the
Applybutton is not activated. - Unable to create entries in Website SSO / Identity Brokering ( SAML)
…ex into admin-ui-issue-2512
|



feat(admin-ui): unused components cleanup and migration to single navigation wrapper (#2512)
Summary
Large-scale cleanup of unused components across the Admin UI and migration from scattered static navigations to a unified navigation wrapper.
Cleanup Overview
A total of 69+ unused components were identified and marked for removal through automated scanning combined with manual grep verification.
Components Targeted for Removal
Gluu Components (14)
Demo / Sample Components (36)
Charts, analytics, dashboards, tables, UI samples and placeholder components.
Internal Components (3)
Layout (1)
Plugin Components (11)
Mostly unused legacy forms, LDAP/SQL config pages, bar chart implementations and unused hooks/schemas.
Core Components (2)
Dashboard Components (4)
CustomBadgeRow, CustomBarGraph, CustomLineChart, CustomPieGraph
Utilities (1)
🧭 Navigation Migration
Goal
Move to a single unified Navigation wrapper instead of multiple static navigations.
Actions:
useAppNavigationhook.navigate()calls withnavigateToRoute()utility.ROUTESconstants.📊 Expected Improvements
⚡ Performance Check
Build time
Bundle size
Impact
🔗 Ticket
Closes: #2512