Skip to content

Let a post override its unfurl image - #666

Merged
madolson merged 1 commit into
valkey-io:mainfrom
madolson:og-image-override
Sep 10, 2026
Merged

madolson merged 1 commit into
valkey-io:mainfrom
madolson:og-image-override

Conversation

@madolson

@madolson madolson commented Sep 10, 2026

Copy link
Copy Markdown
Member

This got accidentally dropped from #651 so the unfurled image isn't right, so fixing that.

This was generated by AI but verified, with love, by a human.

A featured_image is a wide hero banner, which crops badly as an unfurl
card. Adds an og_image extra that takes precedence over featured_image
so a post can ship a purpose-built card.

Signed-off-by: Madelyn Olson <madelyneolson@gmail.com>
@coderabbitai

coderabbitai Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 62a0a643-c28b-44ac-9a06-1877f989bc96

📥 Commits

Reviewing files that changed from the base of the PR and between 3bd5e54 and 362cb0d.

📒 Files selected for processing (1)
  • templates/includes/head.html

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The og:image metadata now prefers page.extra.og_image, then page.extra.featured_image, and finally the default Valkey logo image.

Changes

Open Graph Image Selection

Layer / File(s) Summary
Template image resolution
templates/includes/head.html
The og:image tag now checks page.extra.og_image before page.extra.featured_image and the default logo path.

Merge Risk: ⚪ Minimal · up to 362cb

Posts with an og_image will use it for social unfurls, while existing fallbacks remain unchanged. No merge-blocking risk is identified.


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.

@madolson
madolson merged commit 106ab1c into valkey-io:main Sep 10, 2026
4 checks passed
@madolson
madolson deleted the og-image-override branch September 10, 2026 16:49
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