Skip to content

feat: implement audit detail page with findings view (#32) - #75

Merged
Kami-no-san merged 3 commits into
VeridionLabs:mainfrom
Emmanuel-Ugochukwu1:feat/audit-detail-page
Aug 12, 2026
Merged

feat: implement audit detail page with findings view (#32)#75
Kami-no-san merged 3 commits into
VeridionLabs:mainfrom
Emmanuel-Ugochukwu1:feat/audit-detail-page

Conversation

@Emmanuel-Ugochukwu1

Copy link
Copy Markdown
Contributor

Summary

Implements the audit detail page with a findings view (issue #32).

Changes

  • Add audit detail route (/dashboard/audits/[id]) with finding list, severity chart, and finding cards.
  • Add shared api-helpers.ts for typed API calls (apiGet/apiPost).
  • Update the audits service/controller for detail lookups.
  • Apply Prettier formatting to satisfy CI format:check.

Closes #32

- API: Add PATCH /audits/findings/:id endpoint for finding status updates
  - Validates status against allowed values (OPEN, ACKNOWLEDGED, etc.)
  - Enforces authorization (user must own the audit)

- Web: Create reusable audit components
  - FindingCard: expandable card with status selector, code display, AI summary
  - FindingList: findings with search, severity/status/plugin filters
  - SeverityChart: pie chart showing severity distribution via recharts

- Web: Rewrite audit detail page with real API integration
  - Fetches audit + findings from API with loading/error/empty states
  - Stats grid showing security score, status, findings count, duration
  - Optimistic status updates with revert on failure
  - Active 'Verify on-chain' button with placeholder
  - Link to AI Chat for contextual questions

- Web: Improve audit list page
  - Real API integration with status filtering and search
  - Links to detail pages
  - Proper loading, empty, and error states

- Web: Extract shared API helpers to lib/api-helpers.ts
Consume the raw response shape returned by the backend (matching the
api-client contract) instead of a { success, data } envelope, and read
the veridion_access_token key so auth is shared across pages.
@Kami-no-san

Copy link
Copy Markdown
Contributor

Thank you for your contribution @Emmanuel-Ugochukwu1 complete 10 issues and submit your wallet for reward

@Kami-no-san
Kami-no-san merged commit 6b60511 into VeridionLabs:main Aug 12, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Web: Implement Audit detail page with findings view

2 participants