paritytech / substrate-website

BSD Zero Clause License
3 stars 13 forks source link

Embed Youtube playlists for past seminars instead of static archive page #317

Open sacha-l opened 1 year ago

sacha-l commented 1 year ago

Right now to update the seminars in the Archive page (https://substrate.io/ecosystem/resources/past-seminars/), we need to manually add the titles and description to the JSON file that the page is using. This causes duplication of content, and sometimes the content needs changing with increases a margin of error for having potentially conflicting sources of truth.

I had mentioned this in the past, where my suggestion was to use the YouTube api to pull seminar information from target playlists. But a way more simple and effective solution would simply be to display the playlists in an iframe.

There are now two channels where Seminars live, one is the ParityTech YouTube channel and the other is the Polkadot one. Going forward we will only be uploading new videos to the Polkadot channel. So as part of my suggestion, this Archive page should be separated in two:

  1. The top heading should be for the Polkadot Youtube Channel, i.e. "Substrate Seminar Live Streams (2023)" with the current playlist being populated here
  2. Below, we should have another heading, i.e. "Substrate Seminar Live Streams (2021-2022)" with the different playlists found here.

Would something like this be possible? Thanks for looking into what can be done here.

carinaluise commented 1 year ago

Thanks for submitting this enhancement idea Sacha! Will get a preview link to you asap

sacha-l commented 1 year ago

Thanks for the update here @carinaluise ! One thing to note that may be relevant here is how Events will be published on polkadot.network in the future. I've heard the events team plan to use a whitelabel solution to publish upcoming events as well as (IIUC) past events. Not exactly sure how this may potentially duplicate efforts here just thought I'd flag this.

carinaluise commented 1 year ago

Thanks for flagging that Sacha, I will double check this with Jonny 👍

carinaluise commented 1 year ago

@sacha-l double checked with Jonny and the events whitelabel solution won't effect this so we can go ahead :)

Here is the preview link for you: https://deploy-preview-321--substrate-website.netlify.app/ecosystem/resources/past-seminars/

Let me know if you have any feedback/ extra suggestions for this

sacha-l commented 1 year ago

Looks good ! Could you just update the title to read: Past Seminars

carinaluise commented 1 year ago

Good shout @sacha-l , looks cleaner with the new title: https://deploy-preview-321--substrate-website.netlify.app/ecosystem/resources/past-seminars/

sacha-l commented 1 year ago

Is there a reason why the "Substrate Seminar Live Streams (2023)" section is only rendering one video in that playlist? there should be two. Other than that it looks good to me, great work 💪🏻 🙏🏻

carinaluise commented 1 year ago

Thanks @sacha-l ! Currently we are embedding the playlist in an iframe, so to access all of the videos in the playlist you can click on the top right button next to the copy link, then a dropdown appears with all the videos in that specific playlist. Let me know if this solution works for you 👍

sacha-l commented 1 year ago

Sure that's fine ⚡

carinaluise commented 1 year ago

@jonnysmillie are you happy with this / shall we get this deployed ?

carinaluise commented 1 year ago

@sacha-l we have implemented some slight design tweaks to improve the UI on this a bit - fancy checking out the deploy preview? https://deploy-preview-321--substrate-website.netlify.app/ecosystem/resources/past-seminars/ :)

sacha-l commented 1 year ago

Looks good to me !