Skip to content

feat: add Open Graph metadata for public previews - #3334

Open
tammi-23 wants to merge 1 commit into
mainfrom
feat/open-graph-tags
Open

feat: add Open Graph metadata for public previews#3334
tammi-23 wants to merge 1 commit into
mainfrom
feat/open-graph-tags

Conversation

@tammi-23

@tammi-23 tammi-23 commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

This PR implements #2875
and add Open Graph tags for previews

@dschmidt

dschmidt commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Oh wow, this works? I was always convinced this would only work with server side rendering


/**
* Injects Open Graph tags (https://ogp.me) describing the currently displayed page, most notably
* public links. Only consumers that execute JavaScript pick these up; preview-specific tags for

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

did you do research if crawlers nowadays do execute JS? Otherwise this whole PR would be kind of pointless. 馃檲

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I mean, it can probably be tested easily... just paste a link into whatsapp or any other app that renders previews

Would be good to at least find one that works with your approach

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants