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

Add 2016 recordings #69

Merged
merged 10 commits into from Mar 9, 2017
Merged

Conversation

jasonaowen
Copy link
Contributor

@jasonaowen jasonaowen commented Mar 2, 2017

Update the 2016 recordings page to show the videos in the YouTube playlist!

After looking through the 2014 and 2015 recordings pages, they seem to have some very similar (but not identical) JavaScript. So, here's my plan:

  • extract 2014 talks into a JSON file, and use AJAX to load it
  • extract 2015 talks into a JSON file, and use AJAX to load it
  • refactor to extract common JavaScript into a shared file, used by both 2014 and 2015
  • put 2016 talks into a JSON file
  • update the 2016 recordings page to include the shared script and the 2016 data

Resolves #68 add 2016 recordings page

@lkuper
Copy link
Member

lkuper commented Mar 2, 2017

Maybe @ertyseidohl will have more feedback, but this approach sounds reasonable to me! I say carry on. 👍

The 2015 talk transcripts are named after the author slugs, but the 2014
talks do not. This leads to a minor difference in the construction of
the transcript links. To support extracting the recording page
JavaScript into a separate, shared file, update the 2015 talk data to
include the (partial) filename of the transcript.
Continue to unify the 2014 and 2015 JavaScript by moving the message for
missing videos from code into data.
Now that the only difference between 2015 and 2016 is the transcript
links, make that a paramater that each passes in.
@jasonaowen
Copy link
Contributor Author

Okay, refactoring done! Tomorrow I'll add the 2016 data.

Issue bangbangcon#68 Add 2016 recordings page
Don't show a link if the `authorslug` is set to `false`.
Issue bangbangcon#68 Add 2016 recordings page
Also markdown-ize the other quick links.
@jasonaowen
Copy link
Contributor Author

@lkuper @ertyseidohl I think this is ready to merge! It got... kinda big, but I tried to keep the individual commits easily reviewable.

@jasonaowen
Copy link
Contributor Author

I have it up on my fork, if you want to see the end result: https://jasonaowen.github.io/bangbangcon.github.io/2016/ . The only change in that branch that's not in this PR is deleting the CNAME file.

To test, I checked that I could see the embedded video for each talk, and that the link from the speaker to the speakers page worked correctly. Thank you for making sure all the talks in the program are present and in order in the YouTube playlist - that made it much easier for me to double-check that I'd gotten all the talks!

@ertyseidohl
Copy link
Contributor

👍

@ertyseidohl
Copy link
Contributor

I'll go ahead and merge if nobody has any objections

@lkuper
Copy link
Member

lkuper commented Mar 3, 2017

Looks good to me too! Thanks @jasonaowen.

@lkuper
Copy link
Member

lkuper commented Mar 6, 2017

@ertyseidohl are you going to merge this?

@ertyseidohl
Copy link
Contributor

Whoops - didn't get a notif! Sorry @lkuper @jasonaowen -- merging.

@ertyseidohl ertyseidohl merged commit 3d6d3b1 into bangbangcon:master Mar 9, 2017
@jasonaowen jasonaowen deleted the 2016-talks branch March 10, 2017 00:03
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.

None yet

3 participants