Skip to content

Add photo station management UI - #136

Draft
noobydp wants to merge 2 commits into
tracefinity:mainfrom
noobydp:codex/photo-stations-management-ui
Draft

Add photo station management UI#136
noobydp wants to merge 2 commits into
tracefinity:mainfrom
noobydp:codex/photo-stations-management-ui

Conversation

@noobydp

@noobydp noobydp commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Summary

This adds the first frontend surface for managing reusable photo stations. Users can now see saved stations from the main dashboard, rename or delete them, and open a dedicated station corner editor to adjust the saved paper corners.

The PR includes:

  • station cards on the main dashboard
  • rename and delete actions for stations
  • a station corner editing page
  • frontend API helpers for station CRUD operations
  • frontend types for station data and paper-size handling
  • dashboard navigation affordances for saved station workflows

Dependency

This PR depends on #135.

Because these branches live in a fork and the base repository does not have the stacked base branches, this PR targets main. Until 135 lands, GitHub's diff may include commits from that backend PR as part of the stacked branch history.

Implementation Notes

This PR depends on the backend station APIs from the previous PR.

The management UI is intentionally separate from the capture/reuse flow so reviewers can validate the station data model and station management surface before reviewing the larger trace workflow changes.

The station corner editor reuses the existing paper-corner editing interaction model where possible, instead of introducing a separate editor pattern. Paper size values are shared through the existing PaperSize type so A4, Letter, A3, and Tabloid stay consistent across backend and frontend.

Tests

  • corepack pnpm@10.29.2 --dir frontend lint
  • corepack pnpm@10.29.2 --dir frontend exec tsc --noEmit

Code and PR drafted with Codex

@jasonmadigan jasonmadigan left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

thanks noobydp

Comment thread frontend/src/app/page.tsx Outdated
Comment thread frontend/src/app/page.tsx Outdated
Comment thread frontend/src/app/stations/[id]/corners/page.tsx Outdated
Comment thread frontend/src/app/page.tsx
Comment thread frontend/src/types/index.ts Outdated
Comment thread frontend/src/types/index.ts Outdated
@noobydp
noobydp force-pushed the codex/photo-stations-management-ui branch from 3a7a0e0 to 2197926 Compare August 2, 2026 08:46
@noobydp

noobydp commented Aug 2, 2026

Copy link
Copy Markdown
Contributor Author

Updated this branch after review and rebased it onto current main, still stacked on PR #135.

Changes made in this update:

  • Dashboard loading now lets tools, bins, and projects render even if photo-station loading fails.
  • Station UI is hidden unless the backend reports photo_stations: true from /api/api-keys.
  • Replaced the light-mode amber-300 text with theme-aware warning text that uses amber-600 in light mode and amber-300 in dark mode.
  • Added an in-flight guard for station rename saves and made the success cleanup id-scoped, so a slow save cannot close a different station rename.
  • Kept the frontend types scoped to the management UI layer after the unused backend station_image_path field was removed.

Local validation:

  • corepack pnpm lint using the repo-declared pnpm@10.29.2: 0 errors, 9 existing warnings.
  • corepack pnpm exec tsc --noEmit

Code and PR drafted with Codex

@noobydp
noobydp requested a review from jasonmadigan August 2, 2026 08:56
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.

2 participants