feat: add scan history sidebar to Findings page#435
Conversation
utksh1
left a comment
There was a problem hiding this comment.
Thanks for the Findings scan-history idea. This needs changes before merge: it introduces a separate history sidebar inside Findings that duplicates the Scans page/navigation, uses design tokens/classes that do not match the current SecuScan styling, includes a replacement-character separator in the UI, and replaces the findings list with task-result data without preserving existing filters/review state/loading/error handling. Please align it with the existing design/data flow and add focused tests for selecting a scan.
- Fix design tokens in ScanHistory to match SecuScan styling - Replace replacement-character separator with middle dot - Preserve selectedFindingId reset when switching scans - Add focused tests for ScanHistory scan selection
done |
Fixes #276
Users had no way to browse or reload past scan results from the UI.
Changes:
ScanHistorycomponent that fetches past tasks from the existing/api/v1/tasksendpoint and lists them in a sidebargetTaskResult()No new backend endpoints needed — reuses existing
/tasksand/task/{id}/resultAPIs.