Skip to content

Fixed link preview #269

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
khoroshevskyi merged 2 commits into from
Apr 20, 2026
Merged

Fixed link preview #269

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
709993e
Fixed link preview
khoroshevskyi Apr 20, 2026
35d5305
Fixed link preview
khoroshevskyi Apr 20, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions ui/index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,12 +12,12 @@
<meta property="og:title" content="BEDbase" />
<meta property="og:description" content="BEDbase is a unified platform for aggregating, analyzing, and serving genomic region data." />
<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" />
Copy link

Copilot AI Apr 20, 2026

Choose a reason for hiding this comment

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

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.

Suggested change
<meta name="twitter:card" content="summary" />
<meta name="twitter:card" content="summary_large_image" />

Copilot uses AI. Check for mistakes.
<meta name="twitter:site" content="@shefflab" />
<meta name="twitter:title" content="BEDbase" />
<meta name="twitter:description" content="BEDbase is a unified platform for aggregating, analyzing, and serving genomic region data." />
<meta name="twitter:image" content="https://bedbase.org/bedbase_web.svg" />
<meta name="twitter:image" content="https://bedbase.org/bedbase_web_preview.png" />
</head>
<body>
<div id="root"></div>
Expand Down
Binary file added ui/public/bedbase_web_preview.png
View file
Open in desktop
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
3 changes: 1 addition & 2 deletions ui/worker.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,7 @@ const CRAWLER_USER_AGENTS = [

const API_BASE = 'https://api.bedbase.org/v1';
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`;
Copy link

Copilot AI Apr 20, 2026

Choose a reason for hiding this comment

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

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.

Suggested change
const OG_IMAGE = `${SITE_URL}/bedbase_web_preview.png`;
const OG_IMAGE = `${SITE_URL}/bedbase_web.svg`;

Copilot uses AI. Check for mistakes.

interface Env {
ASSETS: Fetcher;
Expand Down
Loading