nytimes/library

Generalize youtube embeds to support more content

Open

#220 aperta il 16 set 2020

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (148 fork)auto 404
enhancementgood first issuegood for learningneeds more context

Metriche repository

Star
 (1180 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Problem Description

From this https://github.com/nytimes/library/issues/67, it seems like we can embed youtube videos. I would like to be able to embed other content as well. For example, we would like to embed:

  1. Google Drive Videos
  2. Google slides / sheets
  3. Other platforms (for example, our company uses cloudapp for internal video sharing)

Feature

Could we have config settings that allow users to set domains / URL patterns? This way we can decide what type of links we want to parse and how they would be embedded.

We might want to have some embed syntax to avoid automatically embedding everything.

I could try to take a stab at this feature if this is something you think Library should support. It seems like since we already support youtube it would be in scope?

Guida contributor

Direzione di ricerca
Esamina l'implementazione esistente dell'embed di YouTube. Comprendi come vengono analizzati gli URL e generati gli iframe. Esplora come aggiungere un'opzione di configurazione per pattern URL personalizzati. Considera l'uso di un sistema di plugin o di un elenco di regex.
Tech stack
javascript
Dominio
full stack
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
3-5 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70