Skip to content

fix(seo): serve apple-touch-icon.png from public/ (LAC-2795)#36

Merged
lacymorrow merged 1 commit into
mainfrom
lac-2795/fix-apple-touch-icon-404
Jul 14, 2026
Merged

fix(seo): serve apple-touch-icon.png from public/ (LAC-2795)#36
lacymorrow merged 1 commit into
mainfrom
lac-2795/fix-apple-touch-icon-404

Conversation

@lacymorrow

Copy link
Copy Markdown
Contributor

Summary

Fixes LAC-2795: https://bones.sh/apple-touch-icon.png returned 404 even though the site head references it.

Root cause: the 180x180 PNG existed at src/app/apple-touch-icon.png, but Next.js App Router only auto-serves the convention filename apple-icon.pngapple-touch-icon.png in src/app/ is never served. Meanwhile src/config/metadata.ts explicitly links /apple-touch-icon.png.

Fix: git mv src/app/apple-touch-icon.png public/apple-touch-icon.png so the file is served at the exact path the metadata references. No code changes; binary is byte-identical (rename, 100% similarity).

Acceptance criteria

  • curl -sI https://bones.sh/apple-touch-icon.png returns 200 after deploy.

Testing notes

  • Verified the PNG is 180x180 8-bit sRGB via magick identify.
  • Grepped repo: nothing imports the file from src/app/; the only reference is the /apple-touch-icon.png URL in src/config/metadata.ts.
  • Post-merge prod verification with curl -sI will be done before closing the issue.

Found during the LAC-2783 orphan-page crawl. Engineer-only task (implement → merge → deploy → verify).

@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
bones Ready Ready Preview, Comment Jul 14, 2026 10:03pm

Request Review

The head links /apple-touch-icon.png (src/config/metadata.ts), but the file
lived in src/app/ under a name Next.js App Router does not auto-serve
(convention is apple-icon.png), so https://bones.sh/apple-touch-icon.png
returned 404. Move the existing 180x180 PNG to public/ so it is served at
the referenced path.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
@lacymorrow
lacymorrow force-pushed the lac-2795/fix-apple-touch-icon-404 branch from 93882b2 to 9952cc6 Compare July 14, 2026 22:02
@github-actions

Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

@lacymorrow
lacymorrow merged commit 9f788ec into main Jul 14, 2026
6 of 7 checks passed
@lacymorrow
lacymorrow deleted the lac-2795/fix-apple-touch-icon-404 branch July 14, 2026 22:12
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