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

Recently added list, isn't #101

Open
tomlokhorst opened this issue Feb 3, 2016 · 3 comments
Open

Recently added list, isn't #101

tomlokhorst opened this issue Feb 3, 2016 · 3 comments

Comments

@tomlokhorst
Copy link
Contributor

The "recently added" list on the homepage (and consequently the Atom and Twitter feeds) isn't a list of recently added videos.

Instead it is a reverse list of the videos in videos.yaml, which is mostly chronologically sorted. Probably due to the instruction in the readme about editions.yaml.

I'm not entirely sure how to solve this. Perhaps we can add an instruction in the readme about always adding new videos to the bottom of videos.yaml?
I think it would be nice to promote newly posted videos on the homepage, even if they were recorded a while ago.

@chriseidhof
Copy link
Owner

Yeah, I think we can sort the videos by editions, then flatten it and take the 10 most recent... It'd be cleaner, I think. WDYT?

@tomlokhorst
Copy link
Contributor Author

Wouldn't that just keep the videos in the same chronological order it is already in?

I think we need to add the concept of "order of addition to pomo.tv", I don't think that's encoded anywhere currently.

@eelco
Copy link
Collaborator

eelco commented Feb 3, 2016

IMHO this is the current data model showing its weakness. In most databases having a ‘created’ and ‘updated’ field is super easy and handy. Everything but somehow adding a date to each video when it was merged feels like a hack to me.

I still think it’s possible with just using the YAML files, although doing it manually would suck, so it would need a script that runs (just before/when) merging and adds the dates. We could use git blame to add the historic dates.

(As a side note, we could also save which github user added the videos and give them a small credit, some people get motivated by that ;)

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

No branches or pull requests

3 participants