Skip to content

feat: Add report comparison view [Closes #36]#480

Closed
shravanithouta108 wants to merge 6 commits into
utksh1:mainfrom
shravanithouta108:feature/report-comparison-view
Closed

feat: Add report comparison view [Closes #36]#480
shravanithouta108 wants to merge 6 commits into
utksh1:mainfrom
shravanithouta108:feature/report-comparison-view

Conversation

@shravanithouta108

Copy link
Copy Markdown
Contributor

Description

Adds a new /reports/compare page that lets users select two scan reports and compare them side by side, showing new, fixed, unchanged findings and severity changes between scans.

Related Issues

Closes #36

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?

  • Ran 5 unit tests via npm run test — all pass
  • Tests cover: new findings, fixed findings, unchanged findings, empty findings edge case, and severity changes
  • Manually verified UI on desktop and mobile viewports

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.

Signed-off-by: shravanithouta108 <shravanithouta108@gmail.com>
Signed-off-by: shravanithouta108 <shravanithouta108@gmail.com>
Signed-off-by: shravanithouta108 <shravanithouta108@gmail.com>
…ixes utksh1#30)

Signed-off-by: shravanithouta108 <shravanithouta108@gmail.com>
@utksh1 utksh1 added gssoc:invalid Admin validation: invalid for GSSoC scoring level:advanced 55 pts difficulty label for advanced contributor PRs type:feature Feature work category bonus label area:frontend Frontend React/UI work labels Jun 2, 2026

@utksh1 utksh1 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Rejecting this as not reviewable for merge. The PR claims to add report comparison, but the diff includes broad unrelated backend, CI, validation, Dockerfile, and test changes, plus an artifact-like file named ; artifact-check and frontend Docker builds are failing. Please open a clean, scoped PR that only implements the report comparison view and its focused tests, without unrelated merged changes or generated artifacts.

@utksh1

utksh1 commented Jun 2, 2026

Copy link
Copy Markdown
Owner

Closed as invalid for this contribution round. The PR is too broad/unrelated to the stated report-comparison feature and currently fails artifact/build checks. Open a clean scoped PR for report comparison only.

@utksh1 utksh1 closed this Jun 2, 2026
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 gssoc:invalid Admin validation: invalid for GSSoC scoring level:advanced 55 pts difficulty label for advanced contributor PRs type:feature Feature work category bonus label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REPORTS] Add comparison view between two scan reports

2 participants