Closed defvol closed 10 years ago
Veo este issue menor:
El live PR está en http://apertura-staging.herokuapp.com
cc: @diegobeauroyre @urkonn pueden probar que no falla nada?
Si pueden, traten de traer el upstream de master
. Github resuelve los trees con el default branch. Ahi hay muchos commits que son de development y no están testeados aún. En master
todo anda bien y son menos commits. Solo el release estarían mergeando.
Hey, perdón, linkeé el SHA y github lo defaultea sobre el branch development
; no puedo linear a un commit en master
directamente. Cancelen este PR y hagan rebase a nuestro master.
Por otra parte el bug del check que aparece en la lista de artículos se arregla simplemente en lib/sidebar/list-item.jade
comentando las líneas:
.item-badges
- if(citizen.logged() && !!item.voted)
img(src='/boot/images/check.png')
ok, entonces haremos merge con upstream/master
listo, ahora el merge es con upstream/master (hice force push)
➜ debate git:(upgrade-0.3.4) git diff origin/upgrade-0.3.4 upstream/master --name-only | wc
52 52 1381
➜ debate git:(upgrade-0.3.4) git diff origin/upgrade-0.3.4 upstream/development --name-only | wc
96 96 2517
➜ debate git:(upgrade-0.3.4)
:+1:
Persiste el glitch:
Me parece que falta hacer el fix que menciono más arriba.
El upgrade a 0.3.4 no era por el glitch visual. Sino por el error que aparecía en el build del client js que resultaba en mostrar toda la web en blanco sin cargar. El mencionado en #57
le pondré display:none a aside.nav-proposal .navlist li a .item-badges
:+1:
Actualizamos a 0.3.4 para arreglar un bug en el build de assets de component (#57)