search

Adrian8422 / app-pets-react

0 stars 0 forks source link

Review #2 #3

Open Alvax10 opened 2 years ago

Alvax10 commented 2 years ago

Buenas adrian! como va? estuve revisando un poco el desafío y tuve un par de problemitas, te muestro:

1- Acá, a la hora de editar el perfil, veo que tenés ambos campos, asique en vez de pedirle al user que rellene ambos, podrías poner un condicional que pregunte si escribió en el input, y si no lo hizo, user el valor ya guardado

Captura desde 2022-08-15 09-25-27

2- A la hora de cambiar la contraseña tuve este error, por lo cual significa que no me dejó cambiarla, pero sí me dejó el nombre de usuario Captura desde 2022-08-15 09-26-12

Luego, al tratar de loguearme con esa nueva contraseña, me tiró un error y parece que se cayó el back:

Captura desde 2022-08-15 09-26-40

Fijate si podés arreglar estos errrores y mientras sigo viendo el resto de la página! cualquier cosita nos vemos en discord!

Adrian8422 commented 2 years ago

Buenas Alvax como va? Estuve chequeando lo que me comentaste y creeria que ya esta solucionado. Tambien agregué el valor por defecto en los inputs de editar nombre y email. Gracias por la chequeada y espero que salga bien todo. Saludosss

Alvax10 commented 2 years ago

Genial adri! Vuelvo de la facu y lo termino de chequear! 🙌🏻

DenisMPar commented 2 years ago

Hola adrian! ahi esstuve mirando el desafio y me volvio a pasar lo mismo que a alvax, me dejo loguear pero despues de cambiar la contraseña se cayó el back y no anduvo mas. Puede ser que el endpoint tenga un error que no entra en un catch y por eso se te cae el backend.

Screenshot_2

Adrian8422 commented 2 years ago

Hola Denis como va! Acabo de acomodar el endpoint, no tenia aplicado bien catch. Ahora estaba probando la app y me funciona bien, puedo cambiar de contraseña y demás. Cuando puedas pegale una chequeada a ver si se soluciono. Muchas gracias, saludo !

FerDR89 commented 2 years ago

Hola Adri, ¿Cómo va? Perdón la demora en la corrección. Genial que pudiste solucionar lo de la pass, funciona de 10.

En la parte del front, esta page en mobile se rompe. Para mí tenes algún componente que queda por afuera del viewport (no hace a la funcionalidad del desafío pero cerraría todo perfecto). image

A nivel funcional, cuando cargo una mascota, me redirige a "mis mascotas reportadas" y hay una promesa que no se resuelve y rompe. Te dejo la screen:

image

Fijate si en la DB te cree varias mascotas con data vacía porque veo que se están enviando request vacíos al endpoint (https://current-desafio-m-7.herokuapp.com/me/reports-pets) donde reportas la mascota cuando cargas la ubicación:

image

Dale que te falta nada y ya sale! Cualquier duda apóyate en el Discord que te damos una mano