fiorellaeffio / lim-2018-05-bc-core-am-datadashboard Goto Github PK

El proyecto cumple con los requerimientos de funcionalidad y alcanza los porcentajes mínimos de cobertura.

• Html : Super bien la estructura cumple los estilos y pasa el validador , te recomendamos anivel de semántica utilizar html 5 semántico utilizando etiquetas como section / main / form / article.

• Css : Te recomendamos utilizar clases para dar estilos y reutilizarlos.

• Data.js : Buena indentación, te recomendamos mantener un nombre de variables descriptivos.
  En el siguiente código podrían refactorizarlo y retornar directamente el código que almacenas en la variable utilizando el return implícito de las arrow functions.
  
  También podrían declarar las siguientes variables utilizando const ya que no le vuelves a asignar un valor dentro de la función.
  

• Main.js : Recuerda utilizar EC6 en este caso cuentas con variables declaradas utilizando var y te recomendamos utilizar functions expressions implementando EC6.
  Igualmente recuerda dividir tu código en funciones que se encarguen de realizar una determinada tarea. Por ejemplo puedes creer una función que se encargue solo de 'pintar' en tu html y otra que se encargue de realizar las peticiones Ajax de fecth.

• Git / Github : Recuerda trabajar colaborativamente coordinando con tu equipo el uso de pull request. Igualmente no olvides hacer uso de issues, millestones para trackear tu organización y delegar tareas.
  Buen Trabajo, sigue así 👍