joscejaimes / cdmx-2018-06-bc-core-am-cipher Goto Github PK

Josceline,

• Bien por tu README. Buena apreciación UX. Me hubiera gustado que fueras más específica en fotos y en tu investigación con datos, para hacerlo aún más interesante. Destaco tu "Cómo descargar la aplicación" como buena práctica.

• Me hubiera gustado ver más commits y no sólo dos. Hay que ser más específicos con la descripción de tus commits.

• El cipher es funcional y está terminado. Sólo hay varios detalles que quisiera comentarte sobre su desempeño y organización.

• Tus tests no pasan ni hacen render en su index.html. Es porque tienes errores de sintaxis. Revisa en tu cipher.spec.js la línea 18.

Y, cuidado con las comillas. Si usas comillas y necesitas comillas dentro de las comillas, utiliza apóstrofes para hacer la diferenciación. Tenías esto:

Debería ser así:

• Luego, los tests no pasan porque tus funciones no retornan al final un valor. Si yo ejecuto en consola estos dos comandos, regresan undefined. Deberían regresar el valor del cifrado.
  

• Veo que tu mismo cipher.js pinta sobre el HTML. La idea es que en el index.js se pinte el proyecto y en tu cipher.js sólo se describan las funciones. Separar la lógica de la vista.

• Tienes varias etiquetas en tu HTML que no están cerradas. Te comparto donde están todos esos errores.

• Faltó invertir más tiempo en tus estilos CSS. Tienes un error en la línea 4 con una propiedad que no existe.
  

Buen proyecto Josceline. Atento de tus comentarios.