feat: Implement admin dashboard by mdsaad31 · Pull Request #12 · mdsaad31/Excel-Analytics-Platform

mdsaad31 · 2025-07-21T16:26:13Z

This commit introduces a new admin dashboard with the following features:

User management (view, update role, delete)
Public chart management (view, delete)
File management (view, delete)
Analytics overview

The backend has been updated with new admin-only API endpoints, and the frontend includes a new /admin route protected by an AdminRoute component.

This commit introduces a new admin dashboard with the following features: - User management (view, update role, delete) - Public chart management (view, delete) - File management (view, delete) - Analytics overview The backend has been updated with new admin-only API endpoints, and the frontend includes a new /admin route protected by an AdminRoute component.

netlify · 2025-07-21T16:26:20Z

❌ Deploy Preview for excel-data-analytics failed. Why did it fail? →

Name	Link
🔨 Latest commit	`fee582f`
🔍 Latest deploy log	https://app.netlify.com/projects/excel-data-analytics/deploys/687e6a28b091e00008b152de

Copilot

Pull Request Overview

This PR implements a comprehensive admin dashboard feature that provides administrators with centralized management capabilities for users, public charts, files, and analytics. The implementation includes both frontend components for the admin interface and backend API endpoints with proper authentication and authorization.

Adds admin role-based access control with protected routes and middleware
Creates a tabbed admin dashboard interface for managing users, charts, files, and viewing analytics
Implements secure backend admin API endpoints with JWT authentication and role verification

Reviewed Changes

Copilot reviewed 14 out of 15 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
src/pages/AdminDashboard.jsx	Main admin dashboard component with tabbed interface
src/components/auth/AuthContext.jsx	Enhanced to fetch user profile with role information
src/components/auth/AdminRoute.jsx	Route protection component for admin-only pages
src/components/admin/UserManagement.jsx	User management interface with role updates and deletion
src/components/admin/FileManagement.jsx	File management interface for viewing and deleting files
src/components/admin/ChartManagement.jsx	Chart management interface for public chart operations
src/components/admin/Analytics.jsx	Analytics overview displaying system metrics
src/App.jsx	Adds admin route configuration
server/routes/userProfile.js	Adds authentication and authorization to existing endpoints
server/routes/admin.js	New admin API endpoints with role-based access control
server/models/userProfile.model.js	Adds role field to user profile schema
server/middleware/auth.js	JWT authentication middleware
server/index.js	Registers admin router
package.json	Adds express-oauth2-jwt-bearer dependency

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

Copilot · 2025-10-18T09:22:45Z

src/components/admin/UserManagement.jsx

+
+  useEffect(() => {
+    fetchUsers();
+  }, []);


The fetchUsers function should be included in the useEffect dependency array to follow React hooks best practices and prevent potential stale closure issues.

Copilot · 2025-10-18T09:22:45Z

src/components/admin/FileManagement.jsx

+
+  useEffect(() => {
+    fetchFiles();
+  }, []);


The fetchFiles function should be included in the useEffect dependency array to follow React hooks best practices and prevent potential stale closure issues.

Copilot · 2025-10-18T09:22:45Z

src/components/admin/ChartManagement.jsx

+
+  useEffect(() => {
+    fetchCharts();
+  }, []);


The fetchCharts function should be included in the useEffect dependency array to follow React hooks best practices and prevent potential stale closure issues.

Copilot · 2025-10-18T09:22:46Z

src/components/admin/Analytics.jsx

+    };
+
+    fetchAnalytics();
+  }, []);


The fetchAnalytics function should be included in the useEffect dependency array to follow React hooks best practices and prevent potential stale closure issues.

Suggested change

}, []);

}, [getAccessTokenSilently]);

Copilot · 2025-10-18T09:22:46Z

server/routes/admin.js

+    try {
+        const charts = await SavedChart.find({ isPublic: true });
+        res.json(charts);
+    } catch (error) {
+        res.status(500).json({ message: 'Error fetching public charts' });
+    }
+});
+
+router.delete('/charts/:chartId', async (req, res) => {
+    try {
+        const deletedChart = await SavedChart.findByIdAndDelete(req.params.chartId);
+        if (!deletedChart) {
+            return res.status(404).json({ message: 'Chart not found' });
+        }
+        res.json({ message: 'Chart deleted successfully' });
+    } catch (error) {
+        res.status(500).json({ message: 'Error deleting chart' });
+    }
+});
+
+// File Management
+router.get('/files', async (req, res) => {
+    try {
+        const files = await FileHistory.find();
+        res.json(files);
+    } catch (error) {
+        res.status(500).json({ message: 'Error fetching files' });
+    }
+});
+
+router.delete('/files/:fileId', async (req, res) => {
+    try {
+        const deletedFile = await FileHistory.findByIdAndDelete(req.params.fileId);
+        if (!deletedFile) {
+            return res.status(404).json({ message: 'File not found' });
+        }
+        res.json({ message: 'File deleted successfully' });
+    } catch (error) {
+        res.status(500).json({ message: 'Error deleting file' });
+    }


Inconsistent indentation using spaces instead of the established 2-space pattern used throughout the rest of the file.

Suggested change

try {

const charts = await SavedChart.find({ isPublic: true });

res.json(charts);

} catch (error) {

res.status(500).json({ message: 'Error fetching public charts' });

}

});

router.delete('/charts/:chartId', async (req, res) => {

try {

const deletedChart = await SavedChart.findByIdAndDelete(req.params.chartId);

if (!deletedChart) {

return res.status(404).json({ message: 'Chart not found' });

}

res.json({ message: 'Chart deleted successfully' });

} catch (error) {

res.status(500).json({ message: 'Error deleting chart' });

}

});

// File Management

router.get('/files', async (req, res) => {

try {

const files = await FileHistory.find();

res.json(files);

} catch (error) {

res.status(500).json({ message: 'Error fetching files' });

}

});

router.delete('/files/:fileId', async (req, res) => {

try {

const deletedFile = await FileHistory.findByIdAndDelete(req.params.fileId);

if (!deletedFile) {

return res.status(404).json({ message: 'File not found' });

}

res.json({ message: 'File deleted successfully' });

} catch (error) {

res.status(500).json({ message: 'Error deleting file' });

}

try {

const charts = await SavedChart.find({ isPublic: true });

res.json(charts);

} catch (error) {

res.status(500).json({ message: 'Error fetching public charts' });

}

});

router.delete('/charts/:chartId', async (req, res) => {

try {

const deletedChart = await SavedChart.findByIdAndDelete(req.params.chartId);

if (!deletedChart) {

return res.status(404).json({ message: 'Chart not found' });

}

res.json({ message: 'Chart deleted successfully' });

} catch (error) {

res.status(500).json({ message: 'Error deleting chart' });

}

});

// File Management

router.get('/files', async (req, res) => {

try {

const files = await FileHistory.find();

res.json(files);

} catch (error) {

res.status(500).json({ message: 'Error fetching files' });

}

});

router.delete('/files/:fileId', async (req, res) => {

try {

const deletedFile = await FileHistory.findByIdAndDelete(req.params.fileId);

if (!deletedFile) {

return res.status(404).json({ message: 'File not found' });

}

res.json({ message: 'File deleted successfully' });

} catch (error) {

res.status(500).json({ message: 'Error deleting file' });

}

Copilot · 2025-10-18T09:22:46Z

server/routes/admin.js

+    try {
+        const charts = await SavedChart.find({ isPublic: true });
+        res.json(charts);
+    } catch (error) {
+        res.status(500).json({ message: 'Error fetching public charts' });
+    }
+});
+
+router.delete('/charts/:chartId', async (req, res) => {
+    try {
+        const deletedChart = await SavedChart.findByIdAndDelete(req.params.chartId);
+        if (!deletedChart) {
+            return res.status(404).json({ message: 'Chart not found' });
+        }
+        res.json({ message: 'Chart deleted successfully' });
+    } catch (error) {
+        res.status(500).json({ message: 'Error deleting chart' });
+    }


Inconsistent indentation using spaces instead of the established 2-space pattern used throughout the rest of the file.

Suggested change

try {

const charts = await SavedChart.find({ isPublic: true });

res.json(charts);

} catch (error) {

res.status(500).json({ message: 'Error fetching public charts' });

}

});

router.delete('/charts/:chartId', async (req, res) => {

try {

const deletedChart = await SavedChart.findByIdAndDelete(req.params.chartId);

if (!deletedChart) {

return res.status(404).json({ message: 'Chart not found' });

}

res.json({ message: 'Chart deleted successfully' });

} catch (error) {

res.status(500).json({ message: 'Error deleting chart' });

}

try {

const charts = await SavedChart.find({ isPublic: true });

res.json(charts);

} catch (error) {

res.status(500).json({ message: 'Error fetching public charts' });

}

});

router.delete('/charts/:chartId', async (req, res) => {

try {

const deletedChart = await SavedChart.findByIdAndDelete(req.params.chartId);

if (!deletedChart) {

return res.status(404).json({ message: 'Chart not found' });

}

res.json({ message: 'Chart deleted successfully' });

} catch (error) {

res.status(500).json({ message: 'Error deleting chart' });

}

Copilot · 2025-10-18T09:22:47Z

server/routes/admin.js

+    try {
+        const charts = await SavedChart.find({ isPublic: true });
+        res.json(charts);
+    } catch (error) {
+        res.status(500).json({ message: 'Error fetching public charts' });
+    }
+});
+
+router.delete('/charts/:chartId', async (req, res) => {
+    try {
+        const deletedChart = await SavedChart.findByIdAndDelete(req.params.chartId);
+        if (!deletedChart) {
+            return res.status(404).json({ message: 'Chart not found' });
+        }
+        res.json({ message: 'Chart deleted successfully' });
+    } catch (error) {
+        res.status(500).json({ message: 'Error deleting chart' });
+    }
+});
+
+// File Management
+router.get('/files', async (req, res) => {
+    try {
+        const files = await FileHistory.find();
+        res.json(files);
+    } catch (error) {
+        res.status(500).json({ message: 'Error fetching files' });
+    }
+});
+
+router.delete('/files/:fileId', async (req, res) => {
+    try {
+        const deletedFile = await FileHistory.findByIdAndDelete(req.params.fileId);
+        if (!deletedFile) {
+            return res.status(404).json({ message: 'File not found' });
+        }
+        res.json({ message: 'File deleted successfully' });
+    } catch (error) {
+        res.status(500).json({ message: 'Error deleting file' });
+    }


Inconsistent indentation using spaces instead of the established 2-space pattern used throughout the rest of the file.

Suggested change

try {

const charts = await SavedChart.find({ isPublic: true });

res.json(charts);

} catch (error) {

res.status(500).json({ message: 'Error fetching public charts' });

}

});

router.delete('/charts/:chartId', async (req, res) => {

try {

const deletedChart = await SavedChart.findByIdAndDelete(req.params.chartId);

if (!deletedChart) {

return res.status(404).json({ message: 'Chart not found' });

}

res.json({ message: 'Chart deleted successfully' });

} catch (error) {

res.status(500).json({ message: 'Error deleting chart' });

}

});

// File Management

router.get('/files', async (req, res) => {

try {

const files = await FileHistory.find();

res.json(files);

} catch (error) {

res.status(500).json({ message: 'Error fetching files' });

}

});

router.delete('/files/:fileId', async (req, res) => {

try {

const deletedFile = await FileHistory.findByIdAndDelete(req.params.fileId);

if (!deletedFile) {

return res.status(404).json({ message: 'File not found' });

}

res.json({ message: 'File deleted successfully' });

} catch (error) {

res.status(500).json({ message: 'Error deleting file' });

}

try {

const charts = await SavedChart.find({ isPublic: true });

res.json(charts);

} catch (error) {

res.status(500).json({ message: 'Error fetching public charts' });

}

});

router.delete('/charts/:chartId', async (req, res) => {

try {

const deletedChart = await SavedChart.findByIdAndDelete(req.params.chartId);

if (!deletedChart) {

return res.status(404).json({ message: 'Chart not found' });

}

res.json({ message: 'Chart deleted successfully' });

} catch (error) {

res.status(500).json({ message: 'Error deleting chart' });

}

});

// File Management

router.get('/files', async (req, res) => {

try {

const files = await FileHistory.find();

res.json(files);

} catch (error) {

res.status(500).json({ message: 'Error fetching files' });

}

});

router.delete('/files/:fileId', async (req, res) => {

try {

const deletedFile = await FileHistory.findByIdAndDelete(req.params.fileId);

if (!deletedFile) {

return res.status(404).json({ message: 'File not found' });

}

res.json({ message: 'File deleted successfully' });

} catch (error) {

res.status(500).json({ message: 'Error deleting file' });

}

Copilot · 2025-10-18T09:22:47Z

server/routes/admin.js

+router.delete('/files/:fileId', async (req, res) => {
+    try {
+        const deletedFile = await FileHistory.findByIdAndDelete(req.params.fileId);
+        if (!deletedFile) {
+            return res.status(404).json({ message: 'File not found' });
+        }
+        res.json({ message: 'File deleted successfully' });
+    } catch (error) {
+        res.status(500).json({ message: 'Error deleting file' });
+    }
+});


Inconsistent indentation using spaces instead of the established 2-space pattern used throughout the rest of the file.

mdsaad31 requested a review from Copilot October 18, 2025 09:21

Copilot AI reviewed Oct 18, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: Implement admin dashboard#12

feat: Implement admin dashboard#12
mdsaad31 wants to merge 1 commit intomainfrom
feat/admin-dashboard

mdsaad31 commented Jul 21, 2025

Uh oh!

netlify bot commented Jul 21, 2025 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Oct 18, 2025

Uh oh!

Copilot AI Oct 18, 2025

Uh oh!

Copilot AI Oct 18, 2025

Uh oh!

Copilot AI Oct 18, 2025

Uh oh!

Copilot AI Oct 18, 2025

Uh oh!

Copilot AI Oct 18, 2025

Uh oh!

Copilot AI Oct 18, 2025

Uh oh!

Copilot AI Oct 18, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

mdsaad31 commented Jul 21, 2025

Uh oh!

netlify bot commented Jul 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

❌ Deploy Preview for excel-data-analytics failed. Why did it fail? →

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Oct 18, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 18, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 18, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 18, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 18, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 18, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 18, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 18, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

netlify bot commented Jul 21, 2025 •

edited

Loading