LucasCalvetti / lost-pets-app-with-react

https://lost-pets-app-with-react.vercel.app
0 stars 0 forks source link

Feedback #2

Open zapaiamarce opened 1 year ago

zapaiamarce commented 1 year ago

Holas! No puedo ingresar.

image

Tomate el tiempo necesario para revisar todos los flujos bien asi podemos completar la revisión.

Abrazo!

LucasCalvetti commented 1 year ago

Hola Marce gracias por responder, creo que no te funciona porque tengo configurado en el backend del modulo 7 para que checkee si es un mail valido basado en esta funcion:

function validateEmail(email: string) { if (/^\w+([.-]?\w+)@\w+([.-]?\w+)(.\w{2,3})+$/.test(email)) { return true; } return false; }

porque probe con otro mail y me funciona, decis que le setee un alert o algo para que diga "introduzca otro formato de mail" o algo por el estilo?

El vie, 11 nov 2022 a la(s) 14:07, Marcelo Zapaia @.***) escribió:

Holas! No puedo ingresar.

[image: image] https://user-images.githubusercontent.com/1208547/201392467-acb74683-1a1e-4e2d-b376-d26771643e7a.png

Tomate el tiempo necesario para revisar todos los flujos bien asi podemos completar la revisión.

Abrazo!

— Reply to this email directly, view it on GitHub https://github.com/LucasCalvetti/lost-pets-app-with-react/issues/2, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATPUGTEBTNLXYVLWQFSDROLWHZ4LVANCNFSM6AAAAAAR5YZ6PU . You are receiving this because you are subscribed to this thread.Message ID: @.***>

zapaiamarce commented 1 year ago

mmmmm no sé de que me serviría ese alert. Yo necesito loguearme

LucasCalvetti commented 1 year ago

listo marce, ahi ya deberia dejar loguearte

El vie, 11 nov 2022 a la(s) 14:43, Marcelo Zapaia @.***) escribió:

mmmmm no sé de que me serviría ese alert. Yo necesito loguearme

— Reply to this email directly, view it on GitHub https://github.com/LucasCalvetti/lost-pets-app-with-react/issues/2#issuecomment-1311986885, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATPUGTEFL6GTGKNCDTURYHTWH2AV3ANCNFSM6AAAAAAR5YZ6PU . You are receiving this because you commented.Message ID: @.***>

zapaiamarce commented 1 year ago

Ok, si querés pegale una revisada al flujo completo y volvelo a mandar. Abrazo!

LucasCalvetti commented 1 year ago

Perdona Marce estaba a las apuradas antes de entrar en el segundo turno del laburo y quería corregirlo rapido y me olvidé de mandarlo, ahí ya debería aparecerte cómo que lo envié

El vie, 11 de nov. de 2022 3:47 p. m., Marcelo Zapaia < @.***> escribió:

Ok, si querés pegale una revisada al flujo completo y volvelo a mandar. Abrazo!

— Reply to this email directly, view it on GitHub https://github.com/LucasCalvetti/lost-pets-app-with-react/issues/2#issuecomment-1312066791, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATPUGTBM37FN32TOU3PFDUTWH2IFXANCNFSM6AAAAAAR5YZ6PU . You are receiving this because you commented.Message ID: @.***>

zapaiamarce commented 1 year ago

Joya! Ahi lo miro. Abrazo!

FerDR89 commented 1 year ago

Buenas! Algunas cosas a tener en cuenta:

Estaría genial poder cambiar el nombre sin necesidad de tener que cambiar la pass y viceversa

image.

El tema de los estilos y como quedan los componentes (mi recomendación es que en desktop trabajes con un max de 1200px, lo centres y "el resto" quede como margen a los costados porque en monitores grandes no se ve muy estetico).

Mismo la card que te queda pegada al footer y además el footer esta abajo de todo sino que le queda un espacio vacío debajo en blanco

image

image

En mobile necesitas agregar padding hacia los margenes para que no te quede pegado el contenido a los bordes

image

Acá la card debería ser más chica que en desktop y además no te queda separación con el footer

El footer en mobile, en mi opinión, debería ser más chico o tener menos texto.

Cuando voy de "mis mascotas reportadas" a "reportar mascotas" me pasa esto en mobile

image

Y hacia abajo del componente tmb esta roto.

Slds!

LucasCalvetti commented 1 year ago

Gracias fer por el detalle de lo que faltaba y tomarte el trabajo de verlo! creo que ahora esta todo bien, voy a reenviarlo por la pagina de apx