Conversation
Deploying bedhost-ui with
|
| Latest commit: |
35d5305
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://98c5955f.bedhost.pages.dev |
| Branch Preview URL: | https://dev.bedhost.pages.dev |
There was a problem hiding this comment.
Pull request overview
Updates the site’s default social preview metadata to use a PNG image (instead of the current SVG) to improve link preview compatibility across crawlers/social platforms.
Changes:
- Switch default OG image URL in the Cloudflare Worker to
bedbase_web_preview.png. - Switch default
og:imageandtwitter:imageinindex.htmltobedbase_web_preview.png.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| ui/worker.ts | Updates the worker’s default OG image URL used for crawler-specific HTML on dynamic routes. |
| ui/index.html | Updates default Open Graph / Twitter image URLs for the SPA entry page. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| const SITE_URL = 'https://bedbase.org'; | ||
| // Current OG image is an SVG; use a 1200x630 PNG here if one is added for broader social preview compatibility | ||
| const OG_IMAGE = `${SITE_URL}/bedbase_web.svg`; | ||
| const OG_IMAGE = `${SITE_URL}/bedbase_web_preview.png`; |
There was a problem hiding this comment.
OG_IMAGE now points to /bedbase_web_preview.png, but that file does not exist under ui/public/ (only bedbase_web.svg is present). Unless this PNG is being added/hosted elsewhere as part of the deployment, crawlers will get a broken og:image/twitter:image URL. Add the PNG to the static assets (e.g., ui/public/) or update the URL to an asset that is actually shipped.
| const OG_IMAGE = `${SITE_URL}/bedbase_web_preview.png`; | |
| const OG_IMAGE = `${SITE_URL}/bedbase_web.svg`; |
| <meta property="og:url" content="https://bedbase.org/" /> | ||
| <meta property="og:image" content="https://bedbase.org/bedbase_web.svg" /> | ||
| <meta property="og:image" content="https://bedbase.org/bedbase_web_preview.png" /> | ||
| <meta name="twitter:card" content="summary" /> |
There was a problem hiding this comment.
twitter:card is still set to summary even though an explicit preview image is provided. This is inconsistent with the worker/SEO component (which use summary_large_image) and may prevent the large image from rendering in Twitter/X previews for the homepage. Consider switching to summary_large_image here as well.
| <meta name="twitter:card" content="summary" /> | |
| <meta name="twitter:card" content="summary_large_image" /> |
No description provided.