Poner en brightgreen el "ok" cuando se esté controlando con la última versión disponible

[emilioplatzer]

El punto donde creo que hay que poner el control es donde actualmente dice

if("last version"){

(que justamente lo puse para marcar el lugar porque creo que es ahí.

[emilioplatzer]

Nunca comparar con un ID (ej: warn.warning==='older_version_of_qa_control_in_package_json'), hay que comparar una característica (ej: notice).

O sea cuando es gravity notice en vez de mostrar la cantidad de notices se pone ok en verde.

Es más tampoco debería controlarse gravity notice sino por una característica de gravity. Por ejemplo:

        var count={
            error  :{count:0, abr:'err', color:'red'   },
            obs    :{count:0, abr:'obs', color:'yellow'},
            warning:{count:0, abr:'war', color:'orange'},
            notice :{count:0, abr:'not', color:'ffdb99'}, // naranja claro
        }

podría llamarse gravities (en vez de count) y en el caso de notice poner: equalOk. Algo así como

        var gravities = {
            error  :{abr:'err', color:'red'   },
            obs    :{abr:'obs', color:'yellow'},
            warning:{abr:'war', color:'orange'},
            notice :{abr:'not', color:'green', equalOk:true}, 
        }

y usar gravities para saber la abreviación, el color, o si debe reemplazarse por ok. O mejor:

        var gravities = {
            error  :{label:'% err', color:'red'   },
            obs    :{label:'% obs', color:'yellow'},
            warning:{label:'% war', color:'orange'},
            notice :{label:'ok', color:'green'}, 
        }

En ese caso para todos es lo mismo, se pone el label reemplazando "%" por la cantidad (y notice no tiene % así que el reemplazo no hace nada).

[diegoefe]

Y cuándo haya más notice's y me interese sólo el que ser refiere a la versión vieja?

[emilioplatzer]

No se llamarán notices, serán observaciones

On Tue, Apr 19, 2016 at 2:26 PM, Diego Florio notifications@github.com wrote:

T cuándo haya más notice's y me interese sólo la de la versión vieja?

— You are receiving this because you modified the open/close state. Reply to this email directly or view it on GitHub https://github.com/codenautas/qa-control-server/issues/26#issuecomment-212026379

[diegoefe]

Como sea, si los scoring: {notice:1} son observaciones, van a salir en la cucarda final, no se va a ver el "ok" que vos querés, hay que remover el warning correspondiente para que eso pase...

[emilioplatzer]

Acabo de instalarlo. Se ve bien.

[diegoefe]

Cerramos este issue?