Add SBOM vex download to release overview #602
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds SBOM (Software Bill of Materials) download functionality to the project release overview page, allowing users to download SBOM reports in JSON format for selected artifacts.
- Added a download button for SBOM reports with async handling
- Implemented downloadSBOMReport function to fetch and download SBOM JSON files
- Reorganized the UI to include the download button alongside the artifact selector
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| artifacts={releases?.data?.map((r) => r.name) || []} | ||
| /> | ||
| </div> | ||
| <div className="flex relative flex-col"> |
There was a problem hiding this comment.
The CSS class 'relative' appears unnecessary in this context as there are no absolutely positioned child elements. Consider removing it unless it serves a specific purpose.
| <div className="flex relative flex-col"> | |
| <div className="flex flex-col"> |
src/app/(loading-group)/[organizationSlug]/projects/[projectSlug]/overview/page.tsx
Outdated
Show resolved
Hide resolved
src/app/(loading-group)/[organizationSlug]/projects/[projectSlug]/overview/page.tsx
Outdated
Show resolved
Hide resolved
| import { useActiveProject } from "@/hooks/useActiveProject"; | ||
| import { useActiveAsset } from "@/hooks/useActiveAsset"; | ||
| import { useActiveAssetVersion } from "@/hooks/useActiveAssetVersion"; |
There was a problem hiding this comment.
The imports 'useActiveProject', 'useActiveAsset', and 'useActiveAssetVersion' are added but never used in the component. These unused imports should be removed to keep the code clean and avoid confusion.
| import { useActiveProject } from "@/hooks/useActiveProject"; | |
| import { useActiveAsset } from "@/hooks/useActiveAsset"; | |
| import { useActiveAssetVersion } from "@/hooks/useActiveAssetVersion"; |
| }, [completeRiskHistory, mode]); | ||
|
|
||
| const selectedArtifact = useSearchParams()?.get("artifact") || undefined; | ||
| const pathname = usePathname(); |
There was a problem hiding this comment.
The variable 'pathname' is declared using usePathname() but is never used in the component. Consider removing it to avoid confusion.
src/app/(loading-group)/[organizationSlug]/projects/[projectSlug]/overview/page.tsx
Outdated
Show resolved
Hide resolved
…ug]/overview/page.tsx Signed-off-by: Tim Bastin <38261809+timbastin@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Tim Bastin <38261809+timbastin@users.noreply.github.com>
No description provided.