Skip to content
This repository has been archived by the owner on Jul 6, 2021. It is now read-only.

Use new og-image service for episode previews #4

Merged
merged 2 commits into from
Jul 5, 2021

Conversation

rikukissa
Copy link
Member

No description provided.

@rikukissa rikukissa requested a review from anttti July 4, 2021 15:09
Comment on lines -23 to -44
const metaTags = [
{ name: "title", content: title },
{
name: "keywords",
content:
"webbidevaus, web dev, podcast, react, node, typescript, javascript",
},
{
name: "description",
content: description,
},
{ property: "og:title", content: title },
{ property: "og:description", content: description },
{ property: "og:url", content: url },
{ property: "og:image", content: coverUrl },
{ property: "og:type", content: "website" },
{ property: "twitter:title", content: title },
{ property: "twitter:description", content: description },
{ property: "twitter:url", content: url },
{ property: "twitter:image", content: coverUrl },
{ property: "twitter:card", content: "summary_large_image" },
];
Copy link
Contributor

Choose a reason for hiding this comment

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

Missäs nää nyt on?

Copy link
Member Author

Choose a reason for hiding this comment

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

Nää ei ollu ollenkaa käytössä 😬 oli unused variable

Copy link
Member Author

Choose a reason for hiding this comment

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

Käytännössä noi taitaa olla kuitenkin kaikki noissa metatageissa, eli olisko tää peruja jostain vanhasta koodista

Copy link
Contributor

Choose a reason for hiding this comment

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

Joo kattelin tuolla netlifypreviewissä mut unohin tulla kommentoimaan takas, että kyl ne jotenkin sinne tosiaan valuu.

Comment on lines 22 to 24
const coverUrl = `https://og-webbidevaus.vercel.app/${encodeURIComponent(
`**Jakso ${episode.number}**: ${episodeTitleWithoutNumber(episode.title)}`
)}?theme=${episode.number % 2 === 0 ? "light" : "dark"}&md=1&fontSize=100px`;
Copy link
Contributor

Choose a reason for hiding this comment

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

Tätä saa varmaan just sellaseks teemattua kun haluaa? Mietin että vois ainakin jotain väriä laittaa josain kohtaa.

Copy link
Contributor

Choose a reason for hiding this comment

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

Lisäks tuo ** ei näytä boldaavan tekstiä vaan tulee ihan sellaisenaan
image

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah pentele, pitää siirtää noi tähdet ton encodeUriComponent-kutsun ulkopuolelle

@rikukissa rikukissa merged commit 6ceace4 into main Jul 5, 2021
@rikukissa rikukissa deleted the feature/dynamic-preview-images branch July 5, 2021 17:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants