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

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

Open
sacha-l opened this issue Jan 25, 2023 · 12 comments
Open
Assignees
Labels
enhancement New feature or request

Comments

@sacha-l
Copy link
Contributor

sacha-l commented Jan 25, 2023

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.

@dennisparity dennisparity added the enhancement New feature or request label Feb 13, 2023
@carinaluise
Copy link
Contributor

carinaluise commented Feb 20, 2023

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

@sacha-l
Copy link
Contributor Author

sacha-l commented Feb 20, 2023

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
Copy link
Contributor

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

@carinaluise
Copy link
Contributor

@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
Copy link
Contributor Author

sacha-l commented Feb 23, 2023

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

@carinaluise
Copy link
Contributor

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

@sacha-l
Copy link
Contributor Author

sacha-l commented Feb 24, 2023

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
Copy link
Contributor

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
Copy link
Contributor Author

sacha-l commented Mar 7, 2023

Sure that's fine ⚡

@carinaluise
Copy link
Contributor

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

@carinaluise
Copy link
Contributor

@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
Copy link
Contributor Author

sacha-l commented Mar 17, 2023

Looks good to me !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants