RossiniMaximo / dwf-m8-challengue-front

0 stars 0 forks source link

Cosas a corregir #4

Open alexismunoz1 opened 2 years ago

alexismunoz1 commented 2 years ago

Buenas Maximo, ya casi lo tenés, pero antes habría que corregir unas cosas

Consejo: mientras más tiempo te tomes en testear que todas las funcionalidades (reportar una mascota, editar la data de una mascota, eliminar un reporte de una mascota, enviar un reporte vía mail, crear un usuario, cerrar sesión, etc) funcionen bien, más rápido vas a aprobar el desafío, demás esta decir que ese testeo tiene que ser en producción.

En fin, que no decaiga que ya casi lo tenés!

RossiniMaximo commented 2 years ago

Buenas Alex , sisi ahora voy a intentar arreglar el problema de las mascotas cercanas , pasa que estuve probando con distintas APIs y siempre toman mi ubicación digamos me parece, gracias x la revi!!

El lun, 25 abr 2022 a las 10:03, Alexis Muñoz @.***>) escribió:

Buenas Maximo, ya casi lo tenés, pero antes habría que corregir unas cosas

  • Cuando edito una mascota, la imagen lo la edita, es decir, no se intercambia por la nueva, el nombre si y la ubicación no sé. Estaría bueno que la ubicación este presente en la card (esto es opcional)
  • Reporte dos mascotas cerca de mi ubicación y no aparecen en "mis mascotas cercanas", no sé si algo que me sucede a mí o realmente hay algo que falla

Consejo: mientras más tiempo te tomes en testear que todas las funcionalidades (reportar una mascota, editar la data de una mascota, eliminar un reporte de una mascota, enviar un reporte vía mail, crear un usuario y cerrar sesión) funcionen bien, más rápido vas a aprobar el desafío, demás esta decir que ese testeo tiene que ser en producción.

En fin, que no decaiga que ya casi lo tenés!

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

RossiniMaximo commented 2 years ago

Buenas Alexis, ya comitee los cambios , estoy usando la location del browser ahora y los pets se actualizan correctamente , y le agregue la ubicación como me recomendaste

El lun, 25 abr 2022 a las 10:03, Alexis Muñoz @.***>) escribió:

Buenas Maximo, ya casi lo tenés, pero antes habría que corregir unas cosas

  • Cuando edito una mascota, la imagen lo la edita, es decir, no se intercambia por la nueva, el nombre si y la ubicación no sé. Estaría bueno que la ubicación este presente en la card (esto es opcional)
  • Reporte dos mascotas cerca de mi ubicación y no aparecen en "mis mascotas cercanas", no sé si algo que me sucede a mí o realmente hay algo que falla

Consejo: mientras más tiempo te tomes en testear que todas las funcionalidades (reportar una mascota, editar la data de una mascota, eliminar un reporte de una mascota, enviar un reporte vía mail, crear un usuario y cerrar sesión) funcionen bien, más rápido vas a aprobar el desafío, demás esta decir que ese testeo tiene que ser en producción.

En fin, que no decaiga que ya casi lo tenés!

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

alexismunoz1 commented 2 years ago

Genial Maximo, más tarde cuando me desocupe lo miro!

RossiniMaximo commented 2 years ago

Joya muchas gracias!!

On Mon, Apr 25, 2022, 2:20 PM Alexis Muñoz @.***> wrote:

Genial Maximo, más tarde cuando me desocupe lo miro!

— Reply to this email directly, view it on GitHub https://github.com/RossiniMaximo/dwf-m8-challengue-front/issues/4#issuecomment-1108840291, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASD4RGMFRRWZU6WPVQ2UMODVG3H7HANCNFSM5UIQEZOQ . You are receiving this because you commented.Message ID: @.***>