Skip to content

Use published indexed media queries in the wedding gallery - #56

Merged
kvz merged 2 commits into
mainfrom
gallery-lookup
Oct 17, 2026
Merged

kvz merged 2 commits into
mainfrom
gallery-lookup

Conversation

@kvz

@kvz kvz commented Oct 17, 2026

Copy link
Copy Markdown
Member

Why

“Show more” could spend an entire page on photos the wedding gallery immediately hid. Shared video links also duplicated the component's rendition grouping and refused Assemblies with 500 or more results.

Adopt the now-published @transloadit/convex@0.6.0 registry dependency: use its indexed createdAfter before pagination and getStoredMedia for exact links, deleting the app's Assembly scan. One fixed visit clock drives both pagination and rendering. Guest, album, upload-prefix, exact-version and delivery authorization remain app-owned and unchanged; the cutoff is display policy, not deletion or access revocation.

Validation

  • yarn install, yarn dedupe, and published-package resolution check (0.6.0 from npm, not workspace source)
  • yarn check, yarn example:build, yarn verify:local (10 local browser scenarios; cloud acceptance is separate)
  • Regression coverage for hidden history before paging, a fixed cutoff across “Show more”, exact video lookup with more than 500 Assembly results, wrong versions, non-finite cutoffs and existing authorization denials
  • Council review: no issues. Independent Opus 5.5 local authorization review: PASS; two small coverage gaps covered without changing the runtime boundary

The existing main workflow deploys the demo and runs real private-Storage acceptance after merge. No API2 deployment, credential rotation or retention change.

🤖 Claimed by kvz on studio1/convex since 2026-10-17 · resume: codex resume 019fa2ee-f994-79d0-9d50-071bf2cb7ad6

@vercel

vercel Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
convex Ready Ready Preview Oct 17, 2026 7:11pm UTC

Request Review

@kvz
kvz merged commit 44b4bbd into main Oct 17, 2026
12 checks passed

This branch was successfully deployed

1 active deployment
Preview — ed67d897 Deployed Oct 17, 2026 by vercel[bot]
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.

1 participant