admin-dashboard cards#406
Open
sakthiarajapandian27 wants to merge 1 commit into
Open
Conversation
KudratAroraa
requested changes
May 14, 2026
Collaborator
KudratAroraa
left a comment
KudratAroraa
left a comment
There was a problem hiding this comment.
Approved overall, but I would recommend one small fix before merge. The dashboard summary implementation looks good and the KPI cards, loading state, error handling, and overall UI integration are well done. The feature clearly improves the admin homepage and aligns with the intended scope of this PR.
One thing I would suggest updating is the API call inside DashboardHome.jsx. Instead of hardcoding the backend URL there, please use the shared reusable base API configuration from api.js / service layer so it stays consistent with the rest of the dashboard and is easier to maintain. Other than that, the implementation looks good.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Admin Dashboard - Dashboard Summary and Overview Cards
Todos
• Integrated dashboard summary API with admin homepage
• Mapped backend response fields and displayed key metrics using reusable KPI cards
• Improved UI with proper labels, icons, and section structure
• Implemented loading, empty, and error states for better UX
• Ensured consistency with admin design system
• Tested functionality and raised PR
How to test
Run Admin dashboard, and review the dahsboard.
Screenshots and/or Gifs
Associated MS Planner Tasks
Known Issues
none