Skip to content

fix: replace per-page OG endpoint with static opengraph.png#11

Merged
gregfromstl merged 1 commit into
mainfrom
feature/pe-8039-fix-openindex-og-images
Jul 14, 2026
Merged

fix: replace per-page OG endpoint with static opengraph.png#11
gregfromstl merged 1 commit into
mainfrom
feature/pe-8039-fix-openindex-og-images

Conversation

@gregfromstl

Copy link
Copy Markdown
Collaborator

Fixes the broken Open Graph images on the docs site (PE-8039). The per-page /api/og endpoint added in #9 wasn't rendering correctly, so this reverts it in favor of a single static image.

  • Reverts the /api/og takumi endpoint: removes the route, the @takumi-rs dependencies, and the bundled Inter fonts that only existed for it
  • Points ogImageUrl at a static /opengraph.png (new 1200×630 card in site/public/)
  • Keeps the updated site description from feat: per-page Open Graph images #9

Verified by building the site and serving it locally: every page emits <meta property="og:image" content="/opengraph.png"/> and the image serves with a 200.

Reverts the /api/og takumi endpoint from #9 and points ogImageUrl at
a single static 1200x630 opengraph.png instead. Removes the takumi
dependencies and bundled Inter fonts that only existed for it.
@vercel

vercel Bot commented Jul 13, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
openindex Ready Ready Preview Jul 13, 2026 11:23pm

Request Review

@pkg-pr-new

pkg-pr-new Bot commented Jul 13, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/openindex@11

commit: 1aa1bf4

@gregfromstl
gregfromstl marked this pull request as ready for review July 14, 2026 20:27
@gregfromstl
gregfromstl requested a review from r0ohafza July 14, 2026 20:28
@gregfromstl
gregfromstl merged commit 92d7219 into main Jul 14, 2026
8 checks passed
@gregfromstl
gregfromstl deleted the feature/pe-8039-fix-openindex-og-images branch July 14, 2026 20:53
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