Antes que nada, bien por hacer su proyecto muy simple. Fue fácil de usar. Al mismo tiempo, veo varios detalles importantes a arreglar.
No viene en su documentación cómo instalar el proyecto. Básense en otros proyectos como el de Paypal por ejemplo, donde en la sección "Installation" para abajo, te explica cómo puedes hacerlo. Ahí pueden también incluyen tecnologías que están usando, dependencias, etc.
Probé la autenticación con Google y Facebook. Todo bien. También el registro de correo bien. Me gustaría que piensen en el siguiente nivel que es la confirmación del usuario vía correo electrónico. Bien por la validación.
Atención con la ortografía, es deconstruye, no "deconstrulle".
Bien por ejecutar el Eslint. No encuentro problema. Por otro lado, con HTMLHint veo un par de detalles a arreglar con sus etiquetas.
Bien por la cadencia en commits por ambas. Les pido que en todo commit pongan con mayor descripción qué hicieron.
Sean semánticamente elegantes con los nombres de sus archivos. index1.js, index2.js, no me dicen nada.
Les faltaron sus tests.
Aunque el proyecto es flexible, considero que pueden investigar más sobre el concepto de Responsive Web Design y los media queries. No los usan y creo que puede ayudarles más a equilibrar sus proyectos. Por ejemplo, aquí noten cómo se disparejan los renglones.
Conclusión
Su proyecto tiene buena arquitectura y veo mucha mejora en sus funciones. Les pido arreglar los detalles de arriba.
Cualquier duda, acérquense conmigo o con cualquier coach.
Esthela, Elianet.
Antes que nada, bien por hacer su proyecto muy simple. Fue fácil de usar. Al mismo tiempo, veo varios detalles importantes a arreglar.
No viene en su documentación cómo instalar el proyecto. Básense en otros proyectos como el de Paypal por ejemplo, donde en la sección "Installation" para abajo, te explica cómo puedes hacerlo. Ahí pueden también incluyen tecnologías que están usando, dependencias, etc.
Probé la autenticación con Google y Facebook. Todo bien. También el registro de correo bien. Me gustaría que piensen en el siguiente nivel que es la confirmación del usuario vía correo electrónico. Bien por la validación.
Atención con la ortografía, es deconstruye, no "deconstrulle".
Bien por ejecutar el Eslint. No encuentro problema. Por otro lado, con HTMLHint veo un par de detalles a arreglar con sus etiquetas.
Bien por la cadencia en commits por ambas. Les pido que en todo commit pongan con mayor descripción qué hicieron.
Sean semánticamente elegantes con los nombres de sus archivos. index1.js, index2.js, no me dicen nada.
Les faltaron sus tests.
Aunque el proyecto es flexible, considero que pueden investigar más sobre el concepto de Responsive Web Design y los media queries. No los usan y creo que puede ayudarles más a equilibrar sus proyectos. Por ejemplo, aquí noten cómo se disparejan los renglones.
Conclusión
Su proyecto tiene buena arquitectura y veo mucha mejora en sus funciones. Les pido arreglar los detalles de arriba.
Cualquier duda, acérquense conmigo o con cualquier coach.
Gracias y buen esfuerzo.