francisvelueta / cdmx-2018-06-bc-core-am-social-network Goto Github PK

Crear Login Twitter

Crear un post de Recetas

@VaniushaR, MirEnovi y 1 Alterna87

• Su proyecto tiene 2 README.md, lo mejor sería tener solo uno.
• No me deja registrarme, dice"Error No hay registro de usuario correspondiente a este identificador. El usuario puede haber sido eliminado"
• Como buena practica las variables globales se declaran al inicio del documento. (index.js linea 62 y 63)
• Recuerden enfocar su código a test, Les dejo una lectura que espero les ayude https://medium.com/javascript-scene/5-common-misconceptions-about-tdd-unit-tests-863d5beb3ce9.
• Tengan cuidado con sus avences. A mi parecer van un poco atrasadas y se les va a juntar los alcances.
• Pueden mejorar el uso de los issues, oueden poner mas descripción, asignar a responsables por ese issue, y cuando seán resueltos traten de cerrarlos.
• 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.
• La inicialización de firebase debe de ir en un archivo js externo y no dentro del html.
• No es responsive.
• No se rindan, están logrando grandes cosas! 😄

Crear y validar login con Facebook

Francisca, Vania, Mir,

General
Es un proyecto bueno y bien por la ejecución. Tiene detalles importantes a considerar pero en la revisión lo noté completo.

Primero, recuerden siempre indicar en los README.md cómo se instala su proyecto. Es un hábito difícil de crear pero la intención es si hay más personas que quieren probarlo o ustedes en unos años abrirlo para repasar ciertos conceptos, se acuerden qué hicieron.

Eslint

Hice un examen con Eslint para revisar cuáles fueron sus detalles de estilo y fueron 155 alertas, pero sin ningún error.

Recordemos que el Eslint es para mantener un estilo consistente sobre su código. Cuando lo inyecten, deben de correr el programa de Eslint en terminal sobre sus archivos para revisar esos estilos. Pero es para mantener la buena lectura.

JS

Tienen algunos archivos muertos como print.js que no aportan.

Siempre eliminen el código o archivos que no aporten valor a su proyecto. Busquen esa parte limpia (LEAN) de la aplicación todo el tiempo.

Bien por la división lógica/vista con Firebase. Ahí les pido que en su nomenclatura sea todo en inglés. Ya no hagan nada en español en relación a nombrar las variables (sólo los comentarios si gustan).

Veo que utilizaron jQuery. Es importante que al usarlo conozcan un poco más del framework para usarlo adecuadamente como por ejemplo aquí, donde repiten 5 veces $document.ready y podrían abstraerlo en una sola función.

De ahí en fuera, veo una ejecución notable en funciones.

Tests

Veo un buen avance con tests en cuestión de nombrarlos. Lo que seguiría sería probarlos y desarrollar "código test" que trate de romper su código de aplicación.

Hay que terminarlos. Recuerden siempre empezar a los tests. Es difícil al inicio pero al trazar la estrategia del proyecto, consideren cuáles tests deben de crear.

HTML y CSS

Hay varios detalles con respecto al Responsive Design. Cuando utilicen Materialize, aunque les arregla gran parte del Grid, es importante revisar que esos detalles de tamaños entre elementos se tomen en cuenta. Para que no se encimen y pierdan elegancia en su trabajo final.

Las invito a, con un CSS, a tener los elementos generales hasta arriba, luego los IDs(#) y finalmente las clases.

Mezclarlas a la larga les hace muy difícil su mantenimiento.

Conclusión

Buen proyecto. Cuiden mucho la nomenclatura de sus commits. Intenten poner todo en inglés y sigan practicando las fusiones.

Si tienen dudas puntuales, acérquense y las respondemos sin problema.

Crear y validar el login con cuenta de Google

Cuidar muy bien los espacios entre elementos como tipografía, fotografías, logotipo, botones para evitar scrolls innecesarios, alinear bien elementos importantes como CTA, organizar bien los elementos de su página en cuanto a tamaño, alineación, analicen la importancia del contenido para resolver qué es lo importante y darle esa jerarquía.

1.Maquetar la barra y botones de ranking de likes y dislikes
2. Guardar los datos por usuario en Firebase
(Añdir alguna restricción para que no puedan votar muchas veces por un mismo post)

1. Maquetar el botón de favoritos en cada tipo de post, desde el formato.
2. Guardar los datos por cad ausuario en Firebase.