Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Sign up
Appearance settings

CI: add og image generation #2078

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ShubhamOulkar wants to merge 13 commits into expressjs:gh-pages
base: gh-pages
Choose a base branch
Loading
from ShubhamOulkar:og

Conversation

@ShubhamOulkar
Copy link
Member

@ShubhamOulkar ShubhamOulkar commented Oct 8, 2025 •
edited
Loading

@ShubhamOulkar ShubhamOulkar requested a review from a team as a code owner October 8, 2025 13:02
Copy link

netlify bot commented Oct 8, 2025 •
edited
Loading

✅ Deploy Preview for expressjscom-preview ready!

Name Link
🔨 Latest commit ad8869f
🔍 Latest deploy log https://app.netlify.com/projects/expressjscom-preview/deploys/68e67c830417b70008cbcb40
😎 Deploy Preview https://deploy-preview-2078--expressjscom-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

socket-security bot commented Oct 8, 2025 •
edited
Loading

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Added npm/​gray-matter@​4.0.3 99 100 100 82 100
Added npm/​satori@​0.18.3 98 100 100 97 70
Added npm/​sharp@​0.34.4 92 100 100 91 100

View full report

Copy link
Contributor

github-actions bot commented Oct 8, 2025 •
edited
Loading

🚦 Lighthouse Results (Mobile & Desktop)

URL Device Perf A11y Best Practices
/ mobile 🔴 64 🟢 100 🟢 96
/en/blog/posts.html mobile 🟠 87 🟢 96 🟢 96
/en/5x/api.html mobile 🔴 71 🟢 95 🟢 96
/ desktop 🟢 100 🟢 100 🟢 96
/en/blog/posts.html desktop 🟢 97 🟢 96 🟢 93
/en/5x/api.html desktop 🟢 100 🟢 95 🟢 96

@ShubhamOulkar ShubhamOulkar marked this pull request as draft October 8, 2025 14:53
@ShubhamOulkar ShubhamOulkar requested review from bjohansebas and removed request for a team October 9, 2025 02:05
Copy link
Member Author

ShubhamOulkar commented Oct 9, 2025 •
edited
Loading

@bjohansebas can you help me with Netlify preview system? I am not able to test generated OG images in the preview. Local development it works. Generated meta tags are as follows.

<meta property="twitter:image" content="https://expressjs.com/images/og-images/2025-07-31-security-releases.webp">

@ShubhamOulkar ShubhamOulkar marked this pull request as ready for review October 9, 2025 09:49
Copy link
Member

@bjohansebas can you help me with Netlify preview system? I am not able to test generated OG images in the preview. Local development it works. Generated meta tags are as follows.

Are you referring to modifying the way the build is done on Netlify? It’s not necessary for me to do it from Netlify, you can modify it so that it only affects this PR. For example:
https://github.com/bjohansebas/expressjs.com-fork/blob/675297fbf293b971e59ee4731a2dd05adca828ea/netlify.toml

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bjohansebas bjohansebas Awaiting requested review from bjohansebas

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Update opengraph image

AltStyle によって変換されたページ (->オリジナル) /