-
Notifications
You must be signed in to change notification settings - Fork 460
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
Babel #743
Babel #743
Conversation
Solo me queda la duda de como hacer links relativos, por ejemplo aqui: https://github.com/Laboratoria/curricula-js/pull/743/files#diff-7d35de8a9f10aaba6af7bc83f3d45280R176 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gracias por el PR @merunga 🚀 🌟
Antes de continuar con el review, podrías porfa actualizar el PR para que solo incluya cambios del tópico de babel
? Parece que se colaron un par de cambios en archivos de redux
🙈
topics/redux/01-intro/00-basics/README.md
topics/redux/README.md
Esto es una muy buena pregunta... creo que primero nos toca probar a parsear el tópico y ver cómo se interpretan las URLs relativas. En base a eso toca revisar el proceso de parseo y presentación en LMS para que acordemos una forma estándar de crear estos links relativos. cc/ @ivandevp @MaiaRojas |
Hola @lupomontero , en relación a los cambios de redux que comentas, los estaba volviendo a incluir, porque estaban en su PR correspondiente del 27 de setiembre, que nunca se hizo review y seguía abierto... y obviamente esos cambios ya estan en mi repo. Entonces:
🙏 porfa que los PR no se queden dormidos tanto tiempo que es chamba doble. |
@lupomontero algun update de esto? podemos contar con este topic para el inicio de track de FE en lima para el 21 de enero? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gracias por el PR @merunga 🚀 🌠 y disculpa por la demora en contestarte 😰
He dejado varios comentarios, principalmente typos. Quieres que te mande PR con mis sugerencias?
Por otro lado, después de leer las tres lecturas (valga la redundancia), me quedo pensando si quizás sería bueno agregar una breve introducción explicando por qué queremos usar Babel, qué problema resuelve desde nuestro punto de vista como developers. Algo un poco más genérico orientado a un público que nunca ha escuchado hablar de un transpilador antes, o de problemas de compatibilidad con navegadores... quizás algo más de contexto en una lectura de tipo intro. Yo me ofrezco como voluntario a proponer algo. Quizás como un enhancement más adelante para no frenar este PR ahora.
cc/ @Laboratoria/bootcamp-coaches
Como referencia, he subido este tópico al LMS dentro del cohort |
closes #743