Comentarios sobre red social sprint 1

@gloryarz, @EstefaniaMolinaa, @AuraMelisa01.

De los 38 commits, solo uno es de @AuraMelisa01. Recuerden que el trabajo no se tiene que dividir por tema o rama. Se trata de que las 3 sean capaces de escribir y leer todo el código del proyecto.
Creo que los commits de @EstefaniaMolinaa solo son cosas de diseño y alguno del README, aplica lo mismo que el caso anterior.
El archivo .DS_Store, no debería subirse a github.
Cuidado con la indentación en el style.css.
Muy buena separación de responsabilidades en los archivos js. Felicidades!!
En el index.html tienen script en la parte donde solo deben ir links, recuerden, los script van justo antes de cerrar el body.
Los scripts de firebase, para asegurar el correcto funcionamiento deben ser los primeros scripts llamados.
El botón de google, no hace nada?
El de Facebook tampoco.
No se si sea mi lap, pero en modo consola no le puedo dar click a los botones, y en modo celular no se ve completamente responsive, se pierde un pedazo de la pantalla.
Recuerden tener en cuenta y hacer código enfocado a Test, con esto me refiero a la separación de responsabilidades de los archivos js y hacer las funciones puras para poder testearlas. Falta mucha indentación.
Les dejo un link que habla sobre TDD, espero les sirva https://medium.com/javascript-scene/5-common-misconceptions-about-tdd-unit-tests-863d5beb3ce9.
Van por buen camino, solo traten de que las 3 tengan commits similares, y las 3 colaboren en todo el proyecto.
nunca se den por vencidas! 😃.

gloryarz / cdmx-2018-06-bc-core-am-social-network