Skip to content

feat(preprod): Display skipped images in snapshots UI#116041

Merged
NicoHinderling merged 1 commit into
masterfrom
feat/snapshots-skipped-category
May 21, 2026
Merged

feat(preprod): Display skipped images in snapshots UI#116041
NicoHinderling merged 1 commit into
masterfrom
feat/snapshots-skipped-category

Conversation

@NicoHinderling
Copy link
Copy Markdown
Contributor

The backend already sends skipped and skipped_count in the snapshot API
response for selective snapshot uploads, but the frontend was completely
ignoring these fields. This wires up the skipped category so it appears as
its own collapsible section in the sidebar alongside Changed, Removed, Added,
Renamed, and Unchanged.

When a selective snapshot upload only includes a subset of images, the
remaining images that exist in the base but weren't re-uploaded are classified
as "skipped" by the comparison task. Previously these were invisible in the
UI — for example, a build with 392 skipped images would show no indication
of their existence.

Changes:

  • Add SKIPPED to DiffStatus enum and SidebarItem union type
  • Add skipped/skipped_count optional fields to SnapshotDetailsApiResponse
  • Wire skipped images into sidebar sections, status pills, and status counts
  • Add skipped handling to card rendering, list view, and single view
  • Convert two if/else chains to switch statements for consistency

The backend already sends skipped images in the API response for
selective snapshot uploads, but the frontend was ignoring them. Wire
up the skipped category so it appears as its own collapsible section
in the sidebar alongside Changed, Unchanged, etc.

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 21, 2026
@github-actions
Copy link
Copy Markdown
Contributor

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.62%

@NicoHinderling NicoHinderling marked this pull request as ready for review May 21, 2026 18:20
@NicoHinderling NicoHinderling requested a review from a team as a code owner May 21, 2026 18:20
@NicoHinderling NicoHinderling merged commit 8a6b7d4 into master May 21, 2026
72 checks passed
@NicoHinderling NicoHinderling deleted the feat/snapshots-skipped-category branch May 21, 2026 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants