search

meetupjs-ar / meetupjs-ar.github.io

:earth_americas: Sitio oficial de Meetup.js Buenos Aires
https://meetupjs.com.ar
41 stars 38 forks source link

Agregar bundle analyzer en CI #108

Closed durancristhian closed 4 years ago

durancristhian commented 4 years ago

Estaría piola que corramos esto en el CI (yo pensé que se estaba corriendo) y que los archivos esos se publiquen para que podamos ver que es lo que hay en prod

durancristhian commented 4 years ago

Bueno, ya agregué en el README como hacer para obtener el reporte (ANALYZE_BUNDLE npm run build).

Teoría: Sin querer y por como estamos haciendo un require mafioso, se nos da la situación de que en los bundles del blog (listado y cada artículo) ESTAMOS INCLUYENDO TODO EL BLOG ENTERO y en base a una prop del componente que viene desde la page, se muestra una cosa o la otra. Podemos confirmar esto?

Supongo que es porque no puede webpack (o next) interpretar en build time a cual de todos los mdx nos referimos por la prop y entonces mete todo por las dudas.

Por si se necesita contexto, desde el componente página en pages/articulos se le pasa la prop para que el require mafioso funcione.

Screen Shot 2019-11-21 at 23 20 12

@sergiodxa vos tenés idea como solucionar esto?

durancristhian commented 4 years ago

Por otro lado, sigo averiguando como hacer para que el archivo ese esté escondido en la subida a github pages y podamos verlo online con lo que hay en prod.

sergiodxa commented 4 years ago

Tengo que ver cómo lo hacen pero no debería estar pasando, los artículos son MDX no? Deberías poder exportar un layout y que cada .mdx sea su propia página, así hacen en ZEIT y hacia en mi blog personal cuando usaba Next.

Creo recordar que ZEIT tiene un repo que sirve de template para armar un blog con Next y MDX, capaz puedas revisar ahí la implementación?