Master a prod

[nukeador]

No description provided.

[nukeador]

Hay que probar que stage.mozilla-hispano.org está todo perfecto para pasar a prod

/cc @fjaguero @stripTM @jilgue

[stripTM]

@fjaguero para evitar lo del menú que se abre y cierra disponemos de la clase .js en la raíz del documento, de manera que se puede tener un
.js menu { display: none; }
Con esto nos aseguramos que si por lo que sea no carga el js no nos quedamos sin posibilidad de navegar, y para aquellos que si que disponen de js convendría tenerlo lo más arriba del documento que se pueda para evitar parpadeos, así que mi propuesta se subir a un javascript del head esta parte

/* Meter class js al elemento html */
var isSupported = document.getElementById && document.getElementsByTagName;
if (isSupported) {
document.documentElement.className = "js";
}

[fjaguero]

@stripTM perfecto, lo podemos hacer después de todos modos, ya que no es crítico. Revisalo en mobile para ver si quedan más cosas críticas en stage. Así podemos avisar a @nukeador para que con varios vistos buenos haga el merge a producción.

[stripTM]

Totalmente, es algo meramente cosmético.

[nukeador]

Por mi no esperéis, si véis que está estable en stage movedlo a prod cuando queráis, simplemente hay que ejecutar en el server el script para hacer pull en prod, que lo puede hacer cualquiera de tecnico@

[fjaguero]

@willyaranda @stripTM Échenle un ojo y de última podemos quedar para hacer el deploy a cierta hora y así puedo echarle un ojo bien. Si puede ser mañana martes genial.

[stripTM]

¿que sería? ¿probar los foros en stage?

[fjaguero]

Es revisar que el responsive del Wordpress, foro y Wiki funcionen bien. Quizás hay algún detalle que arreglar, pero la cuestión es que no haya nada crítico. A priori está ok, pero por eso decía que si quieres me puedes decir una hora y me conecto para revisarlo a fondo en prod.

[fjaguero]

Lo dejo mergeado listo para el deploy.

[stripTM]

el menú superior que en resoluciones pequeñas es un bocadillo (#menu-nav), siempre aparece visible y el toggle no hace nada. ¿esto es así?

[fjaguero]

@stripTM ¿Lo probaste desde un terminal movil y no te funciona?

[fjaguero]

Si cargas la página con la resolución pequeña debería funcionarte.

[stripTM]

Ya veo, es porque no está pensado para reescalar la ventana, la solución no sería complicada es quitar esto:
// Shows the menu only on mobile devices
if (windowWidth < 768) {
menu.classList.add('close');
}

Y añadir dentro de function showMenu(el)
esto:
menu.classList.add('close');

[fjaguero]

¿En ese caso cuándo se aplicaría la clase close? Por lo que veo showMenu() se lanza cuando se hace click. Había que agregar closeal HTML directamente.

[stripTM]

Iba a subirlo a mi repo pero hice el push directamente aquí 38d5886

[fjaguero]

Ok, ¿ves algo más que se rompa para poder hacer el merge a prod?

[stripTM]

El error
TypeError: jQuery(...).fancybox is not a function stage.mozilla-hispano.org:70

¿lo tenéis controlado y cuando se pase a prod no habrá problemas?
Creo que es el que rompe el carrusel.

[fjaguero]

Sí, si no recuerdo mal está controlado y es ese. No me acuerdo exactamente por qué venía (es tema plugin, está claro). De última se puede hacer un git reset --hard HEAD~1 para volver al anterior en la rama prod.

[fjaguero]

Ya está creado el #160 para trackear el error del carrousel de stage.

[fjaguero]

Ping a esto cuando pueda alguien de técnico

[nukeador]

Ejecutado /root/bin/prod-deploy con éxito

[fjaguero]

Ok, ya veo. Por alguna razón no se está cargando lo que está en stage, ni siquiera la hoja responsive.css.

[fjaguero]

No está cargando nada del reponsive ni el meta viewport.

[fjaguero]

Ya está fixed, era la cache de Wordpress.