NotificationsYou must be signed in to change notification settings
Fork11k
Star94.8k

www(chore): clarify image paths#41451

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.

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

Already on GitHub?Sign in to your account

Jump to bottom

Open

dnywh wants to merge10 commits intomaster

base:master

Choose a base branch

fromdnywh/chore/blog-image-frontmatter

Open

www(chore): clarify image paths#41451

dnywh wants to merge10 commits intomasterfromdnywh/chore/blog-image-frontmatter

+960 −841

Conversation

Copy link

Contributor

dnywh commentedDec 18, 2025•
edited
Loading

What kind of change does this PR introduce?

Frontmatter name change.

What is the current behavior?

We repeatedly mistakethumb forimage and visa versa, meaning the wrong images are used for Open Graph and in-site thumbnails on blog posts. Events and case studies use the same naming convention too.

What is the new behavior?

These two bits of frontmatter are renamed for clarity:

Blog posts:imgSite +imgSocial

That mapping for blog posts:

thumb is nowimgSite
image is nowimgSocial

These two remain the same:

Events:thumb
Case studies:imgSite

Thewww/README.md file has been expanded to clarify all of the above. It now also provides instructions on image optimisation.

To test

A lot of files were touched here. Please help make sure:

The CMS works as intended. This is thebiggest unknown.
All blog posts render the correct image as their on-site thumbnail and Open Graph image. You can test the latter by firing up a draft iMessage. Online Open Graph services like Facebook cache images, so aren’t reliable.
All events render their correct images.
All case studies render their correct images.

dnywh added5 commits

December 18, 2025 14:15

fixes

74dc0de

best practices

feb68e0

fixes

cd13baa

rename

d4e5faa

rename

da3cd0f

Copy link

vercelbot commentedDec 18, 2025•
edited
Loading

The latest updates on your projects. Learn more aboutVercel for GitHub.

Project	Deployment	Review	Updated (UTC)
zone-www-dot-com	Ready	Preview,Comment	Dec 18, 2025 5:59am

7 Skipped Deployments

Project	Deployment	Updated (UTC)
studio	Ignored	Dec 18, 2025 5:59am
cms	Skipped	Dec 18, 2025 5:59am
design-system	Skipped	Dec 18, 2025 5:59am
docs	Skipped	Dec 18, 2025 5:59am
studio-self-hosted	Skipped	Dec 18, 2025 5:59am
studio-staging	Skipped	Dec 18, 2025 5:59am
ui-library	Skipped	Dec 18, 2025 5:59am

Copy link

Contributor

coderabbitaibot commentedDec 18, 2025•
edited
Loading

Important

Review skipped

More than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review.

192 files out of 299 files are above the max files limit of 100. Please upgrade to Pro plan to get higher limits.

You can disable this status message by setting thereviews.review_status tofalse in the CodeRabbit configuration file.

✨ Finishing touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branchdnywh/chore/blog-image-frontmatter

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

Copy link

supabasebot commentedDec 18, 2025

This pull request has been ignored for the connected projectxguihxuzqibwxjnimxev because there are no changes detected insupabase directory. You can change this behaviour inProject Integrations Settings ↗︎.

Preview Branches by Supabase.
Learn more aboutSupabase Branching ↗︎.