Skip to content

BYOC post: lead with what it is, MCP-style hero with VPC outline - #70

Merged
NicholasKissel merged 2 commits into
mainfrom
NicholasKissel/byoc-post-diagrams
Sep 15, 2026
Merged

NicholasKissel merged 2 commits into
mainfrom
NicholasKissel/byoc-post-diagrams

Conversation

@NicholasKissel

Copy link
Copy Markdown
Member

Follow-up to #67.

  • Rewrite the intro around the security perimeter and outbound-only link; drop the Cloud/Self-host/BYOC comparison bullets
  • Replace the deployment-flow diagram up front with "What you get" and dashboard deploy steps; move the docs' overview diagram under "How it works"
  • New hero in the Introducing lockup style (ink Rivet badge + title, brand-colored AWS/GCP/Kubernetes/Terraform tiles inside a dashed "Your VPC" outline). Renderer: pnpm render-byoc-hero -- --output-dir <dir> --vpc [--frames <dir>]
  • Hero is served from hero.png (already on R2 and verified) to escape the stale Cloudflare cache on image.png

@railway-app
railway-app Bot temporarily deployed to rivet-website / website-pr-70 September 15, 2026 05:31 Destroyed
- Rewrite intro around the security perimeter and outbound-only link
- Replace the deployment flow diagram up front with What you get and
  dashboard deploy steps; move the overview diagram under How it works
- New hero renderer in the Introducing lockup style (+ --vpc, --frames)
- Hero now served from hero.png to escape the poisoned image.png cache

Amp-Thread-ID: https://ampcode.com/threads/T-01a0a1ba-571f-72d8-a85e-924a49ba90b7
Co-authored-by: Nicholas Kissel <nicholas@rivet.gg>

@the-company-company the-company-company Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 1 medium-severity finding

Reviewed commit aaf2e6c.

published: "2026-09-15"
category: changelog
image: { file: "image.png?v=2" }
image: { file: "hero.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.

🟠 Medium · Point the post at the uploaded hero

The new renderer writes the blog asset as image.png, and the launch workflow uploads and verifies that exact key before setting image: true. This override instead makes the published page and social metadata request hero.png, for which this change has no producer or prescribed upload, so the hero will be broken unless someone performs an undocumented separate upload.

Use the default image: true (or file: "image.png") so the post resolves the renderer and upload output.

@NicholasKissel
NicholasKissel force-pushed the NicholasKissel/byoc-post-diagrams branch from aaf2e6c to 1f230dc Compare September 15, 2026 05:33
@railway-app
railway-app Bot temporarily deployed to rivet-website / website-pr-70 September 15, 2026 05:33 Destroyed

@the-company-company the-company-company Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟠 1 medium-severity finding

Reviewed commit 1f230dc.

published: "2026-09-15"
category: changelog
image: { file: "image.png?v=7" }
image: { file: "hero.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.

🟠 Medium · Point the post at the uploaded hero

The renderer writes the blog asset as image.png, and the launch workflow uploads and verifies that exact key before setting image: true. This override instead makes the published page and social metadata request hero.png, for which this change has no producer or prescribed upload, so the hero will be broken unless someone performs an undocumented separate upload.

Use the default image: true (or file: "image.png") so the post resolves the renderer and upload output.

@NicholasKissel
NicholasKissel merged commit a79d7f5 into main Sep 15, 2026
4 checks passed
@NicholasKissel
NicholasKissel deleted the NicholasKissel/byoc-post-diagrams branch September 15, 2026 05:36
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