Skip to content
This repository has been archived by the owner on Jul 21, 2019. It is now read-only.

Program should support links to slides and video #600

Closed
aaronblythe opened this issue Oct 9, 2017 · 16 comments
Closed

Program should support links to slides and video #600

aaronblythe opened this issue Oct 9, 2017 · 16 comments

Comments

@aaronblythe
Copy link
Contributor

In 2016 when we finished we provided links at directly on the program page:

https://www.devopsdays.org/events/2016-kansascity/program/

Note the "(slides) (video)" for each talk.

Now that we can add the markdown for embedding slides and video to the specific page for the speaker/talk. It would be nice to also be able to provide these links for the yaml for the program.

@aaronblythe
Copy link
Contributor Author

Since there is no embedding trickery, I think this could be as simple as using video and slides in the data/events/year-city.yml.

Then the update to themes/devopsdays-theme/layouts/program/single.html would simply need this in the section for {{- else if or (eq .type "talk") (eq .type "workshop") -}} and {{- else if eq .type "ignite" -}}

                              {{- if .video -}}
                              {{- if ne .video "" -}}
                                <a href="{{ (printf "%s" .video) }}"> (video) </a>
                              {{- end -}}
                              {{- end -}}
                              {{- if .slides -}}
                              {{- if ne .slides "" -}}
                                <a href="{{ (printf "%s" .slides) }}"> (slides) </a>
                              {{- end -}}
                              {{- end -}} 

@aaronblythe
Copy link
Contributor Author

@aaronblythe
Copy link
Contributor Author

hrmm... maybe they are more in sync than I originally thought. I will update here.

@mattstratton
Copy link
Member

Rather than adding yet another data field, I think we can just detect to see if the talk/ignite has something set for video or slide fields and then display the link.

I’m on thumbs right now and going to bed, but I can work on it tomorrow night. I would prefer to avoid adding more configuration when we already have the info available.

@mattstratton
Copy link
Member

I’m assuming that the links to “video”, etc will go to the talk page and not to the YouTube/speakerdeck, etc page.

@mattstratton
Copy link
Member

And finally, if you want to compare/contrast, always check the version of the theme in the toml file at the root of the theme. I cut a release of the theme not too long ago but I’m pretty sure it was when we were having hiccups with making changes to devopsdays-web so I never pushed it.

@aaronblythe
Copy link
Contributor Author

Makes sense, I originally tried to get the links from the individual speaker page. I assume that is how the title is scraped? Is that what {{ $.Scratch.Get .title }} is doing?

The way I have this now, the link is to leave the site and go to YouTube/Vimeo/Slideshare/Speakerdeck/etc. Are you saying you would prefer that they go to the page that the speaker - title links to? I am fine with that, then this really just becomes a marker that video/slides are on that page, so... maybe an image for "video" and "slides" would be appropriate?

@mattstratton
Copy link
Member

My issue with the offsite link is that a talk could have more than one slide link - I don’t want to be opinionated about where they go.

Although linking back to the talk page is what it already does anyway.

I just really, really don’t want to add even more configuration data. I guess I’m not 100% convonved that users wont click on the talk name to find the videos/slides. Maybe we just add a little badge next to the ones that have something in the video or slide frontmatter so it’s known they exist ... which is what I think you meant by the ‘image’.

@aaronblythe
Copy link
Contributor Author

I think we are in agreement that (correct me if I am wrong):

  • No new attributes, so we will read the YouTube/Vimeo/Slideshare/Speakerdeck/etc. from the individual program page
    • If the YouTube/Vimeo setting are there (and not empty) then show a "video camera" icon
    • If the Slideshare/Speakerdeck setting are there (and not empty) then show a "slideshow" icon
  • In all cases this will just link to the page as the "speaker - title" already does.

I am willing to do this work over the next couple days.

What I need is some assistance on is how to read the YouTube/Vimeo/Slideshare/Speakerdeck/etc. from the individual program page. From above, that is what my question of how is the title being pulled? If I understand this, I think I can take a swing at the rest.

@mattstratton
Copy link
Member

Take a look at this section:
https://github.com/devopsdays/devopsdays-theme/blob/master/layouts/program/single.html#L87-L91

{{ $.Scratch.Get (printf "%s-speaker" .title) }}
 -
 {{ $.Scratch.Get .title }}

Basically, inside of there and that section, you can us $.Scratch.Get .VALUE to get the value of anything.

So for example, you should be able to get the youtube value with $.Scratch.Get .youtube

@bridgetkromhout
Copy link
Contributor

I just really, really don’t want to add even more configuration data. I guess I’m not 100% convonved that users wont click on the talk name to find the videos/slides. Maybe we just add a little badge next to the ones that have something in the video or slide frontmatter so it’s known they exist ... which is what I think you meant by the ‘image’.

I'm 100% with @mattstratton here - and I also want to avoid cluttering up people's program pages unless they opt in to this. I, for one, wouldn't want a lot of duplicate icons on the program of a completed event.

@aaronblythe
Copy link
Contributor Author

I am confused... wouldn't the "opt-in" option to be filling out the video and slides on the data/events page like I have it? If these are not filled out then there is nothing extra on the screen. If there is an opt-in the there would be need to be another configuration object even if it is simply some binary flag. My understanding is that any other configuration is really, really not wanted.

I am not certain what users would do once they navigate to the site. We would need to see analytic data to determine that. I would rather not speculate.

What I do know is that as someone who is asked how to get to the videos, last year I always gave the devopsdays.org site for the KC program since the links were there. If the links are not on the program (or some representation that the videos can easily be found), then I will just give out our youtube link to our channel or playlists. I am fine with this, however I am willing to work together on a solution that is configurable for those that want this functionality and those that don't.

Given the current state, my communication to the attendees and our community that I will send out in the next day or two will be the youtube links instead of the devopsdays program page.

@mattstratton
Copy link
Member

I thought about this in the car. Opt-in will be front matter on the program page MD for a field like “displayMedia” or something. It can then default to displaying the first video it sees (YouTube or Vimeo) and slide type, etc.

I think that accomplishes everytinf.

@mattstratton
Copy link
Member

You also can edit the content on the program page right now in your markdown to say CLICK ON A TALK NAME TO SEE SLIDES AND VIDEO” as a workaround.

@mattstratton
Copy link
Member

And I should totally add some google tag tracking on links to slideshare, YouTube, etc from the talk pages to see what the engagement is like. But that’s forward going.

@mattstratton
Copy link
Member

I also am planning on trying to knock this out later this evening.

mattstratton added a commit to devopsdays/devopsdays-web that referenced this issue Sep 4, 2018
**Implemented enhancements:**

- Allow URL link on "custom" program type [\#611](devopsdays/devopsdays-theme#611) (special thanks to @chriscoffee for this one!)
- Program should support links to slides and video [\#600](devopsdays/devopsdays-theme#600)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants