Skip to content
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

Opengraph #134

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Opengraph #134

wants to merge 3 commits into from

Conversation

drmowinckels
Copy link
Member

This is related to #110.

I've added an initial R script that scrapes OG information from URLs provided.

One problem I see is that when running this, not all sites have OG information set. So we will need to deal with that somehow too.

@drmowinckels drmowinckels requested a review from a team as a code owner November 29, 2024 19:08
@drmowinckels drmowinckels requested review from SoyAndrea and removed request for a team November 29, 2024 19:08
Copy link

Building test-site now! The build will take a few minutes, but look at its progress in the website repo.

Copy link

Building test-site now! The build will take a few minutes, but look at its progress in the website repo.

@drmowinckels
Copy link
Member Author

a positive note to this approach, is that we will also get a confirmation on the URL being reachable (how I discovered Cara has moved her posts from blog/ to posts/ on her site).

@rladies-global
Copy link

Test build failed 😭 Check out the build logs.

@drmowinckels drmowinckels marked this pull request as draft December 1, 2024 11:16
@cosimameyer
Copy link
Contributor

Let me know if you want me to look into it and play around with it :) (I wasn't sure because it's still a draft and wanted to double-check :))

@drmowinckels
Copy link
Member Author

Yes please do @cosimameyer ! I'm busy for the next two weeks and won't have time to look more into it. But I came up with this pretty quick, and thought we could build on it.

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.

3 participants