curriculum icon indicating copy to clipboard operation
curriculum copied to clipboard

Incluye Vite como un bundler en Social Network

Open unjust opened this issue 2 years ago • 6 comments

Con Firebase 9 no podemos agregar los globales de CDN para tener firebase en el proyecto. Ahora usamos import para usar los partes de firebase - y para hacer eso ellos recomiendan el uso de un bundler.

Aun Social Network no esta configurado con un bundler. Puede ser que no tuvimos tiempo para hacerlo o quiza no queremos introducirlo porque requiere configuración y es un concepto adicional para ellas. Ahora estamos usando urls en los modulos como describen aqui: Screen Shot 2022-08-05 at 2 53 56 PM

Pero este me parece "hacky" y también hace un pelo más complejo los tests y estructura de archivos.

Pienso que podemos agregar Vite a este proyecto con minima o nada de configuración para que el proyecto puede usar import normal de module firebase.

Recuerdo que @oxfist propone Vite para mejorar algunos proyectos y pienso que seria bueno para Social Network para arreglar este pain point. Opiniones?

unjust avatar Aug 05 '22 20:08 unjust

Creo que puede ser una buena opción... @unjust, te animarías a enviar un PR para ver cómo se verían los cambios en el boilerplate y README.md?

lupomontero avatar Aug 08 '22 17:08 lupomontero

cc/ @mfdebian

lupomontero avatar Aug 08 '22 17:08 lupomontero

Sería demasiado cool solucionar eso @unjust y sé que @oxfist estará feliz de que incorporemos Vite en algún proyecto! :smile:

mfdebian avatar Aug 08 '22 19:08 mfdebian

Hola! Tomé de base la POC que hice para Cipher y la adapté al proyecto Social Network aquí 👉🏼 https://github.com/oxfist/social-network-vite

Esta base contiene:

  • Jest
  • Linters: ESLint, Stylelint y HTMLHint
  • Prettier
  • Despliegue a GitHub Pages con gh-pages

Lo que creo que faltaría para validar la prueba de concepto es agregar Firebase e implementar un flujo básico como autenticación, aunque con solo imprimir la configuración obtenida de Firebase creo que bastaría. Creo que la próxima semana podría agregar eso al repo.

Espero esto pueda servir de base para la modificación propuesta aquí por @unjust 👍🏼

oxfist avatar Aug 12 '22 15:08 oxfist

Gracias @oxfist. Dime si quieres que haga la prueba con firebase. La semana que viene es un poco ocupada para mi con Pre Admision pero puedo despues.

Pregunta - necesitamos todos los linters (mas que ESLint) y Prettier? Son reqisitos o algo asociado con Vite? Me pregunto si debe ser decision a ellas a agregarlos a su proyecto.

unjust avatar Aug 12 '22 15:08 unjust

@unjust adelante, ningún problema si quieres ver la parte con Firebase 🙌🏼

En cuanto a los linters y Prettier, ninguno es requerido por Vite. Solo los agregué para ser consistente con lo que la base del proyecto tiene actualmente (aunque no creo que sea una mala idea) y Prettier pues pienso que es buena idea incentivar a las estudiantes a usarla.

Creo que ahí hay una buena discusión sobre qué herramientas deberían venir sugeridas en los proyectos, por qué y qué esperaríamos que las estudiantes puedan decidir por su cuenta considerando los OAs.

oxfist avatar Aug 12 '22 16:08 oxfist