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

#72: Added list of AmsterdamPHP videos #82

Merged
merged 1 commit into from
May 6, 2015

Conversation

pascaldevink
Copy link
Member

What

Implementation of #72, building on #77. Added a list of AmsterdamPHP videos from YouTube, replacing the list of blog posts.

How it looks

screen shot 2015-04-18 at 21 05 52

@WyriHaximus
Copy link
Member

👍 👍 This looks pretty neat. Although the video sorting seems off...

@pascaldevink
Copy link
Member Author

That's actually how it was uploaded. The October talk came in late 😕

@WyriHaximus
Copy link
Member

Ah that explains! 👍

@rdohms
Copy link
Member

rdohms commented Apr 18, 2015

What data does youtube provide? Can we try to separate date/title? Regexp should solve the title, but can we extract the date directly?

@WyriHaximus
Copy link
Member

This method on the API is called and it returns a list of playlistItems.

@pascaldevink
Copy link
Member Author

YouTube provides a separate date, but the date that we currently display is in the title. We can separate those, but that means that the title should stay in this particular format.

@pascaldevink
Copy link
Member Author

One option would be to drop everything in the title up to the colon (:) and put in the date YouTube provides. The thing is that publication date doesn't say much. The currently displayed date is the date of the recording. I'm guessing that's more important to people than the publication date.

@rdohms
Copy link
Member

rdohms commented Apr 19, 2015

Yeah ideally we should reg-exp the date out of the title and use just the month... but it is "fragile" to some extent

@rdohms rdohms mentioned this pull request May 5, 2015
@pascaldevink pascaldevink merged commit 0add19f into AmsterdamPHP:master May 6, 2015
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