Skip to content

fix(website): eryxon.eu publication path for OG image + pricing route#888

Closed
SheetMetalConnect wants to merge 3 commits into
v0.6from
website-engineer/ery-282-eu-website-only-clean
Closed

fix(website): eryxon.eu publication path for OG image + pricing route#888
SheetMetalConnect wants to merge 3 commits into
v0.6from
website-engineer/ery-282-eu-website-only-clean

Conversation

@SheetMetalConnect
Copy link
Copy Markdown
Owner

Summary

Website-only publication split for ERY-282 targeting https://eryxon.eu.

Changes in this PR:

  • add global social image meta tags in docs head override (og:image, twitter:image)
  • add default social image asset at website/public/social/og-default.png
  • add /pricing/ docs route content for the requested publication surface
  • include heartbeat verification artifact for .eu

Why split

PR #884 is blocked by unrelated native/app CI; this isolates website publication scope so it can ship independently.

Verification

  • cd website && npm run build passes locally
  • Live checks on https://eryxon.eu captured in:
    • scripts/evidence-handoff/ERY-282-2026-05-27-eryxon-eu-verification.md

Remaining blocker

Production deploy from this runtime remains blocked without CLOUDFLARE_API_TOKEN for wrangler deploy.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 27, 2026

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

Project Deployment Actions Updated (UTC)
eryxon-flow Ready Ready Preview, Comment May 27, 2026 8:38pm

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 27, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
eryxon-flow-website 0d74db5 Commit Preview URL

Branch Preview URL
May 27 2026, 08:38 PM

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 27, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 48a812a8-b657-49eb-86eb-1acf4a21c3fc

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch website-engineer/ery-282-eu-website-only-clean

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 992f61c797

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

const detectedLocale = Astro.currentLocale ?? DEFAULT_LOCALE;
const currentLocale = isLocale(detectedLocale) ? detectedLocale : DEFAULT_LOCALE;
const pathname = Astro.url.pathname;
const defaultSocialImage = `${siteUrl}/social/og-default.png`;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Fix the mislabeled OG image asset

The new og:image URL points to /social/og-default.png, but the committed website/public/social/og-default.png starts with JPEG/JFIF magic bytes (ff d8 ff e0 ... JFIF) rather than a PNG signature. In deployments that set Content-Type from the .png extension, social crawlers can receive image/png for JPEG bytes and fail to render the preview this change is meant to add; please either re-encode the asset as PNG or rename/reference it as .jpg.

Useful? React with 👍 / 👎.

<meta property="og:locale:alternate" content={lang} />
))}
<meta property="og:image" content={defaultSocialImage} />
<meta property="twitter:image" content={defaultSocialImage} />
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Use the Twitter meta name attribute

This adds the Twitter image as property="twitter:image", but Twitter/X card metadata is expected under a name="twitter:image" meta tag. On pages where Starlight does not already emit a name="twitter:image", the newly configured global social image may still be ignored by Twitter/X crawlers, so the publication path can remain without the intended preview image.

Useful? React with 👍 / 👎.

@SheetMetalConnect
Copy link
Copy Markdown
Owner Author

Blog lane updated per board directive:\n\n- Added one publish-ready current post: /articles/latest-development-and-road-to-v0-7/\n- Added deterministic code-generated OG/social SVG assets under website/public/social/blog/latest-development-and-road-to-v0-7/\n- Added /blog/latest-development-and-road-to-v0-7/ redirect to the publish route\n- Build passes locally (website)\n\nApproval gate: do not merge/publish until Luke explicitly replies with approval verb on this PR.

@SheetMetalConnect
Copy link
Copy Markdown
Owner Author

CMO queue update applied: current full-publish candidate is now /articles/why-eryxon-flow-moved-to-apache-2-0/.\n\nIncluded in this PR update:\n- publish-ready article content + metadata\n- deterministic code-generated OG/social SVG fallback assets\n- /blog/why-eryxon-flow-moved-to-apache-2-0/ publish redirect to article route\n- build passes with both /blog// and /articles// rendered\n\nApproval gate remains: no merge/publish until Luke posts explicit approval verb on this PR.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0d74db5db9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +21 to +22
- [Shift handoff with remarks](/guides/shift-handoff-remarks/) documents the operator handoff path.
- [v0.6 release checks](/guides/v06-release-checks/) defines concrete go or no-go checks before expanding rollout.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Replace the proof links that 404

These two proof links point to /guides/shift-handoff-remarks/ and /guides/v06-release-checks/, but those docs pages are not present in website/src/content/docs and a repo-wide rg only finds the slugs in this new article. When readers follow the article’s “public and verifiable” evidence, both links will 404, so either add the referenced docs or link to existing proof pages.

Useful? React with 👍 / 👎.

@SheetMetalConnect
Copy link
Copy Markdown
Owner Author

Board reset from Luke via Hermes.

This PR is removed from the publish approval lane. Do not merge.

Reason:

  • Blog/publication work must be coordinated by CEO before board review.
  • One post = one PR.
  • Preserve existing content; do not drop/rewrite unrelated content.
  • Every post PR must include verified final image assets before Hermes asks Luke.
  • Luke is the board, not dev QA; do not send bundled/mixed PRs for individual review.

CEO coordination issue: Paperclip ERY-323.
A corrected single-post, image-complete PR may be opened after CEO coordinates CMO / Content Lead / Website Engineer / CTO.

@SheetMetalConnect
Copy link
Copy Markdown
Owner Author

Closed by Hermes per board reset: superseded by CEO-coordinated blog lane reset in Paperclip ERY-323. Do not reopen as bundled PR; replace with one post per PR, preserved content, verified images.

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