Alvax10 / Lost_pets_react

0 stars 0 forks source link

Los datos de la mascota no se editan correctamente #9

Open alexismunoz1 opened 2 years ago

alexismunoz1 commented 2 years ago

Alva, cuando se editan los datos de una mascota reportada, solamente una vez pude modificar los datos de la misma, luego no hubo manera de modificar los mismos, intente refrescando la page pero tampoco funciona

Alvax10 commented 2 years ago

Uhh que raro, y eso puede ser problema del front, no? Porque lo probé en postman y me edita de 10 todas las veces

El vie., 25 de marzo de 2022 12:16 p. m., Alexis Muñoz < @.***> escribió:

Alva, cuando se editan los datos de una mascota reportada, solamente una vez pude modificar los datos de la misma, luego no hubo manera de modificar los mismos, intente refrescando la page pero tampoco funciona

— Reply to this email directly, view it on GitHub https://github.com/Alvax10/dwf-m8-desafio/issues/9, or unsubscribe https://github.com/notifications/unsubscribe-auth/APKSAAVHUZOFTC44PRSYCMTVBXKD3ANCNFSM5RUOEKYQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

Alvax10 commented 2 years ago

che @alexismunoz1 lo acabo de probar en producción y me dejó editar las mascotas 2 veces cada una, ahora están como editada3 y editada4, tmb cambié la foto y me anduvo perfecto, te tiró algun error en la network?

Alvax10 commented 2 years ago

ahora la cambié a "editada5" para probar porr tercera vez y me funcionó, puedo saber que hiciste que te falló?

Marcosreuquen commented 2 years ago

Buenas!! Estuve viendo por qué lado podía venir ese error. Y me parece que está en el payload de lo que manda el front al back para pedir el update. Me seguía pasando que no se modificaba hasta que vi que sólo se modifica si cambias todos los datos. Entonces, entiendo que el front pide todos los datos de la mascota/formulario y los manda al back, entonces alguno viaja como null y rompe el back de forma que no modifica. En el request debería viajar sólo el dato que cambió o todos los datos anteriores más el que cambió pisado.

Alvax10 commented 2 years ago

Buenas @Marcosreuquen !! AHHHH genialll! entonces ahora me hago un controller para que se pueda modificar la cantidad de información que quieras que se modifique sin que se rompa el back, gracias!

Alvax10 commented 2 years ago

Buenas @Marcosreuquen y @alexismunoz1 , ya pude arreglar el error! ahora podes modificar cualquier dato de la forma que quieras, ya sea uno solo, dos o todos 😄

alexismunoz1 commented 2 years ago

Buenas Alva, ahí lo estuve probando y solamente actualiza la imagen, la ubicación y el nombre de la mascota no. Pobre de todas las maneras posibles, llenando todos los campos, de a un campo a la vez y no hay manera Captura de pantalla de 2022-03-28 18-39-12 Captura de pantalla de 2022-03-28 18-39-34 Captura de pantalla de 2022-03-28 18-37-54

Alvax10 commented 2 years ago

Buenas @alexismunoz1 ! que raro che, literalmente me fui a probar recién la app y mirá, te muestro que me anduvo: Captura de pantalla de 2022-03-28 19-10-42

Captura de pantalla de 2022-03-28 19-10-50

Captura de pantalla de 2022-03-28 19-10-57

Ahí edité solamente el nombre ed "editada6" a "editada7" y me lo hizo perfectamente

Alvax10 commented 2 years ago

Captura de pantalla de 2022-03-28 19-14-21

Ahí modifiqué solo la foto y le puse la misma que a "aaaaaaaaa"

Alvax10 commented 2 years ago

Pero ahora que lo veo, el unico que si que no se edita solo, es la geoloc, asique voy a ver que onda ese, cuando lo probé en prod, antes de mandar devuelta el desafío, me dejaba

Alvax10 commented 2 years ago

Ahí arreglé esos pequeños errores que saltaban, ahora lo pruebo y para revisar que no falle nada y aviso que onda! :D