Skip to content

feat(ui) : add analytics dashboard for vulnerability and scan insights#420

Open
parneetbrar234-svg wants to merge 5 commits into
utksh1:mainfrom
parneetbrar234-svg:feat-analytics-dashboard
Open

feat(ui) : add analytics dashboard for vulnerability and scan insights#420
parneetbrar234-svg wants to merge 5 commits into
utksh1:mainfrom
parneetbrar234-svg:feat-analytics-dashboard

Conversation

@parneetbrar234-svg
Copy link
Copy Markdown
Contributor

@parneetbrar234-svg parneetbrar234-svg commented May 29, 2026

Description

Added a new Security Analytics Dashboard to provide visual insights into vulnerability trends, scan statistics, and target risk analysis.

Features Added

  • Analytics page (/analytics)
  • Vulnerability trends line chart
  • Severity distribution pie chart
  • Most vulnerable targets bar chart
  • Scan success vs failure analytics
  • Summary metric cards
  • Time range filters (7d / 30d / 90d)
  • Analytics navigation item
  • Responsive dashboard layout

Notes

Currently the dashboard uses mock analytics data and is structured for future backend integration and real-time updates.

Related Issues

Closes #376

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

How Has This Been Tested?

  • Verified Analytics page renders correctly
  • Verified route navigation to /analytics
  • Verified charts render successfully using Recharts
  • Verified responsive layout on different screen sizes
  • Verified time range filter buttons work correctly
  • Ran npm run typecheck
  • Ran npm run test

Checklist

  • My code follows the code style of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.

@parneetbrar234-svg
Copy link
Copy Markdown
Contributor Author

@utksh1 plz review my PR.

@utksh1 utksh1 added level:advanced 55 pts difficulty label for advanced contributor PRs type:feature Feature work category bonus label type:testing Testing work category bonus label area:frontend Frontend React/UI work labels May 30, 2026
Copy link
Copy Markdown
Owner

@utksh1 utksh1 left a comment

Choose a reason for hiding this comment

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

Thanks for the dashboard work. I can't merge this as-is because the production Analytics route is wired into navigation with hard-coded sample metrics (, , , and ). That would show fictional security data to users. Please connect the page to real backend/API data, or keep it behind an explicit demo/storybook-only surface. Also restore the AppShell formatting so the navigation arrays and mobile nav remain consistent.

@utksh1
Copy link
Copy Markdown
Owner

utksh1 commented May 30, 2026

Clarifying the requested change: the production Analytics route currently uses hard-coded sample arrays: vulnerabilityTrend, severityData, targetRisk, and scanStats. Please replace those with real API/backend data, or keep the mock dashboard out of production navigation.

@parneetbrar234-svg
Copy link
Copy Markdown
Contributor Author

@utksh1 Removed the Analytics dashboard from production navigation and routing because it currently relies on mock data. The dashboard remains in the codebase and can be reintroduced once real backend/API analytics data is implemented. Also restored AppShell mobile navigation consistency.

Copy link
Copy Markdown
Owner

@utksh1 utksh1 left a comment

Choose a reason for hiding this comment

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

Re-reviewed after the latest update. This still cannot merge while the production Analytics route/nav show hard-coded sample security metrics instead of real API/backend data. Please replace the mock arrays with real data or keep the dashboard out of production navigation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:frontend Frontend React/UI work level:advanced 55 pts difficulty label for advanced contributor PRs type:feature Feature work category bonus label type:testing Testing work category bonus label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Implement Interactive Global Analytics Dashboard

2 participants