Comentarios red social sprint1

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

https://sakurabravo.github.io/cdmx-2018-06-bc-core-am-social-network/src/index.html

0 stars 2 forks source link

Comentarios red social sprint1 #26

Open Jonhks opened 6 years ago

Jonhks commented 6 years ago

@zabarca, @SakuraBravo , @YocelinGR

Porque los inputs que no son de contraseña, se ven como contraseña? -
En la parte de unirme, cual es la funcionalidad de un boton de "Enviar" y otro "Submit"?
En el muro para celulares no se ve el input para comentar y el input que se ve, no envia nada.
Al ingresar con google no me cambia de pagina.
Al ingresar con facebook no me cambia de pagina.
Faltan validaciones de los inputs.
Lo mejor es que un proyecto solo tenga un README principal.
Tienen 2 carpetas de imagenes.
Buena estructura de carpetas
Buena dispersión de responsabilidades del proyecto.
Buen numero de commits y buena participación del equipo aunque @zabarca tiene pocos commits.
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.
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, continúen así!! 😃

YocelinGR commented 6 years ago

Gracias John, a nombre de Zareth, Sakura y mio hacemos constar que recibimos el Issue y que trabajaremos lo antes posible en realizar las mejoras. Saludos cordiales.

zabarca commented 6 years ago

Gracias John! Haré más commits, en el pasado sprint fueron pocos porque teníamos problemas de conflictos y no quedaban solucionados aunque se hicieran las correcciones en el editor o en Github, así que los enviaba a Sakura o a Yocelin por Slack. Envío evidencia por Slack.

SakuraBravo commented 6 years ago

Sip! De hecho en algunos commits puse que eran aportaciones de Zareth :)

Igual podemos mandar fotos de pantalla

El El jue, 26 de julio de 2018 a la(s) 22:07, zabarca < notifications@github.com> escribió:

Gracias John! Haré más commits, en el pasado sprint fueron pocos porque teníamos problemas de conflictos y no quedaban solucionados aunque se hicieran las correcciones en el editor o en Github, así que los enviaba a Sakura o a Yocelin por Slack. Envío evidencia por Slack.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/SakuraBravo/cdmx-2018-06-bc-core-am-social-network/issues/26#issuecomment-408298047, or mute the thread https://github.com/notifications/unsubscribe-auth/AllQYgTdH7bmnqQ_FVrX_A7vyzBLBYv1ks5uKoP9gaJpZM4Viakx .

-- Sakura Bravo Diseñadora Integral sakurabravo.com http://sakurabravo.com