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.

Sign up for GitHub

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

Jump to bottom

Community videos scraping and list page #2441

Open
wants to merge 35 commits into
base: main
Choose a base branch
from
Open

Community videos scraping and list page #2441

wants to merge 35 commits into from

Conversation

cuihtlauac
Copy link
Collaborator

@cuihtlauac cuihtlauac commented May 21, 2024 •
edited
Loading

  • Data type and scraping for YouTube video MRSS
  • YouTube page
  • Add UI
    image

@cuihtlauac cuihtlauac marked this pull request as draft May 21, 2024 09:33
sabine
sabine reviewed May 21, 2024
View reviewed changes
Copy link
Collaborator

@sabine sabine left a comment

Choose a reason for hiding this comment

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

Hey Cuihtlauac, this is awesome!

Would it make sense to have this as part of the OCaml Planet feed instead of a separate page? I.e. have the OCaml Planet have two content types: blog_post and video?

@cuihtlauac
Copy link
Collaborator Author

I posted it here precisely to provoke that kind of discussion :-) I reused the resource design; it was just easy on the first try. I have no strong views on how it should look or where it should go.

@cuihtlauac cuihtlauac force-pushed the scrape-youtube branch 2 times, most recently from ef52283 to 5e35733 Compare May 27, 2024 16:04
@cuihtlauac cuihtlauac force-pushed the scrape-youtube branch 4 times, most recently from 63f9b6f to 215c3d3 Compare June 7, 2024 07:05
@cuihtlauac cuihtlauac marked this pull request as ready for review June 10, 2024 14:27
@cuihtlauac
Copy link
Collaborator Author

I'm sorry @christinerose, I have to revert the changes you made to youtube.yml. This file is automatically generated from content fetched from other sources :-(

gpopides and others added 15 commits July 2, 2024 10:38
add deserialization
346c3d8
add serialization
d743c1c
swap them
aa48e40
@gpopides @sabine
Update data/cookbook/deserialise-from-json/00-deserialize-yojson.ml
548864e 
Co-authored-by: sabine <6594573+sabine@users.noreply.github.com>
@gpopides @sabine
Update data/cookbook/deserialise-from-json/00-deserialize-yojson.ml
4c8f3ee 
Co-authored-by: sabine <6594573+sabine@users.noreply.github.com>
@gpopides @sabine
Update data/cookbook/deserialise-from-json/00-deserialize-yojson.ml
d9c3e71 
Co-authored-by: sabine <6594573+sabine@users.noreply.github.com>
@gpopides @sabine
Update data/cookbook/serialise-to-json/00-serialize-yojson.ml
4706434 
Co-authored-by: sabine <6594573+sabine@users.noreply.github.com>
@gpopides @sabine
Update data/cookbook/serialise-to-json/00-serialize-yojson.ml
800727f 
Co-authored-by: sabine <6594573+sabine@users.noreply.github.com>
@gpopides @sabine
Update data/cookbook/serialise-to-json/00-serialize-yojson.ml
f373a7d 
Co-authored-by: sabine <6594573+sabine@users.noreply.github.com>
Data type and scraping for YouTube video MRSS
d43e063
YouTube page
5486d30
Add UI
fa39123
Formatting
5123666
Per source filters
4bd629a
Only use ocaml as scrape filter
3c3a783
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sabine sabine sabine left review comments

@christinerose christinerose christinerose approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@cuihtlauac @sabine @christinerose @gpopides