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

Agregar feature para transformar links de youtube, vimeo, etc #65

Open
lupomontero opened this issue Jan 25, 2023 · 1 comment
Open
Assignees

Comments

@lupomontero
Copy link
Member

En v5.x necesitamos todavía implementar el feature que analice y reemplace links a proveedores como YouTube, Vimeo, Loom, Google Slides, etc en el body de los archivos markdown analizados. Este feature ya existe a día de hoy en la versión v4.x, pero todavía no se he implementado en el PR de la rama v5.x.

En la versión actual el análisis de los links se hace usando JSDOM, pero en la nueva versión lo ideal sería que no necesitemos agregar JSDOM y que directamente trabajemos con el árbol que produce el parser de Markdown que estamos usando ahora.

Acá los linkProviders que soporta la versión actual de curriculum-parser:

const linkProviders = [

@unjust
Copy link
Contributor

unjust commented Jan 26, 2023

Estaba trabajando un poco en esto anoche. Para probar mi codigo, uso un test que llama parseReadmes con un argumento de directorio de fixtures. El detalles es parseReadmes busca markdowns se llama README en si, entonces tuve que renombrar uno de los READMEs en los fixtures (README-with-youtube-short-link.md -> README.md)

Existe una mejor manera? Voy cambiando los fixtures para cada README suelto tiene un directorio (para todos pueden tener el nombre README), hasta que encuentro mejor manera.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants