chore: drop gallery/admin/guestbook + reorder socials#40
Merged
kaiiiichen merged 3 commits intomainfrom Apr 25, 2026
Merged
Conversation
Order contact icons as Email, LinkedIn, GitHub, X, Telegram so the primary contact channel leads. Co-authored-by: Claude <noreply@anthropic.com> Made-with: Cursor
Drop the entire Supabase-backed photo gallery (`/gallery`, `/admin`, `/admin/gallery`) along with the `/auth/callback` OAuth route that only served the admin sign-in flow. Update the navigation, drop the now unused `@supabase/ssr` dependency, simplify CI (no need for placeholder Supabase env vars), and bring README, CLAUDE, SECURITY, and `.env.example` in line with the trimmed surface area. Co-authored-by: Claude <noreply@anthropic.com> Made-with: Cursor
Drop POST /api/guestbook (no UI consumed it) and the lib/supabase.ts anon-client helper that only existed to back the guestbook insert. Update README, CLAUDE, SECURITY, and .env.example so the only remaining Supabase usage — the lastfm now-playing route writing listening_history / listening_stats with the service role key — is described accurately. Co-authored-by: Claude <noreply@anthropic.com> Made-with: Cursor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
app/gallery/,app/admin/,app/admin/gallery/, and theapp/auth/callback/Supabase OAuth route that only served admin sign-in. Drops theGallerylink from the nav. Removes@supabase/ssr(no longer used) and the now-redundant CI placeholder Supabase env vars (every remaining Supabase client is constructed lazily inside a function body, sonext buildno longer needs them).POST /api/guestbook(no UI consumed it) along with thelib/supabase.tsgetSupabaseAnon()helper that only existed to back the insert. The lastfm now-playing route — which uses its owncreateClient+ service role key — is the only remaining Supabase consumer.README.md,CLAUDE.md,SECURITY.md, and.env.exampleto remove every gallery / admin / guestbook reference, drop the unusedNEXT_PUBLIC_SITE_URLandNEXT_PUBLIC_SUPABASE_ANON_KEYenv vars, simplify the CI section, fix the repo-layout tree, and update the Common Issues / Forking / External Integrations tables.Test plan
npm run lint— only pre-existing warnings, no new onesnpm run typecheck— clean (after wiping.next/cached types from deleted routes)npm run test— 5/5 vitest tests passnpm run build— green; route table confirms/gallery,/admin,/admin/gallery,/api/guestbookall goneMade with Cursor