RossiniMaximo / dwf-m8-challengue-front

0 stars 0 forks source link

Issues #2

Open MatiToledo opened 2 years ago

MatiToledo commented 2 years ago

Hola Máximo como estas ? Estuvo viendo la pagina y note un par de cositas

Se que son una banda de cosas y es un bajón, pero no son difíciles de corregir y la page va a mejorar un montón. Suerte !! Cualquier cosita háblame !

RossiniMaximo commented 2 years ago

Hola Matias como te va?. Ya estoy trabajando en arreglar esos detalles , apenás los soluciono te aviso! Gracias x la revisión , saludos!

El lun, 18 abr 2022 a las 16:14, Matias Toledo @.***>) escribió:

Hola Máximo como estas ? Estuvo viendo la pagina y note un par de cositas

-

El menú me aparece en medio de la pagina no si esta así apropósito o no pero queda raro, también estaría bueno que cuando elijo una de las opciones del menú y cambia de page el menú se cierre solo y no tener que cerrarlo con el clase . También no debería aparecer el cartel de log out si no inicie sesión todavía [image: image] https://user-images.githubusercontent.com/82173133/163860946-49ea2f6b-08b9-448e-b3fc-537d96a3ebbf.png

A la hora de iniciar sesión si pongo una contraseña incorrecta la pagina no me avisa nada y si ahí vuelvo al menú queda como si hubiera iniciado sesión con mi mail cuando en realidad todavía no inicie sesión

A la hora de reportar mascotas no se porque se reportan varias veces, recién se reporto 3 veces xd, y también como que el botón de search de mapbox hace que se envié el reporte a la api (creo)

Cuando edite el reporte de una mascota, funciono mal y le puse el nombre de otra mascota que ya había creado antes (no se porque será) y también estaría bueno que cuando entre a update pet la page de update pet ya tenga cargada la inflo de la mascota que voy a reportar tipo que en el input nombre aparezca ya el nombre con el que fue reportada la mascota

A pesar de haber eliminado las mascotas que reporte de mas de mis reportes, cuando doy mi ubicación siguen apareciendo como perdidas

Algunas cositas generales: Estaría bueno ponerle algún estilo de carga a los botones que hacen llamados a la api que demoran cosa que el usuario sepa que esta pasando algo, también redirigir a otra pagina cuando termina de realizarse esos procesos y podes darle información de lo que pasa al usuario con alerts ( https://developer.mozilla.org/es/docs/Web/API/Window/alert)

Se que son una banda de cosas y es un bajón, pero no son difíciles de corregir y la page va a mejorar un montón. Suerte !! Cualquier cosita háblame !

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

RossiniMaximo commented 2 years ago

Hola mati buen dia , ya commitee los cambios , hubo alguna cosa que no me salio , como por ejemplo el problema al reportar los pets, no se porque pero hay que hacer doble click , estoy tratando de encontrar la solución, ya hable a discord también, pero bueno , puse un cartel que avisa que hay que hacer doble click en ese botón. Saludos!

El lun, 18 abr 2022 a las 16:14, Matias Toledo @.***>) escribió:

Hola Máximo como estas ? Estuvo viendo la pagina y note un par de cositas

-

El menú me aparece en medio de la pagina no si esta así apropósito o no pero queda raro, también estaría bueno que cuando elijo una de las opciones del menú y cambia de page el menú se cierre solo y no tener que cerrarlo con el clase . También no debería aparecer el cartel de log out si no inicie sesión todavía [image: image] https://user-images.githubusercontent.com/82173133/163860946-49ea2f6b-08b9-448e-b3fc-537d96a3ebbf.png

A la hora de iniciar sesión si pongo una contraseña incorrecta la pagina no me avisa nada y si ahí vuelvo al menú queda como si hubiera iniciado sesión con mi mail cuando en realidad todavía no inicie sesión

A la hora de reportar mascotas no se porque se reportan varias veces, recién se reporto 3 veces xd, y también como que el botón de search de mapbox hace que se envié el reporte a la api (creo)

Cuando edite el reporte de una mascota, funciono mal y le puse el nombre de otra mascota que ya había creado antes (no se porque será) y también estaría bueno que cuando entre a update pet la page de update pet ya tenga cargada la inflo de la mascota que voy a reportar tipo que en el input nombre aparezca ya el nombre con el que fue reportada la mascota

A pesar de haber eliminado las mascotas que reporte de mas de mis reportes, cuando doy mi ubicación siguen apareciendo como perdidas

Algunas cositas generales: Estaría bueno ponerle algún estilo de carga a los botones que hacen llamados a la api que demoran cosa que el usuario sepa que esta pasando algo, también redirigir a otra pagina cuando termina de realizarse esos procesos y podes darle información de lo que pasa al usuario con alerts ( https://developer.mozilla.org/es/docs/Web/API/Window/alert)

Se que son una banda de cosas y es un bajón, pero no son difíciles de corregir y la page va a mejorar un montón. Suerte !! Cualquier cosita háblame !

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

Marcosreuquen commented 2 years ago

Buenas, Máximo! Me sumo al issue de Mati porque creo que tiene que ver.

Reporté una mascota y no me apareció en el inicio como que estaba cerca. Ahora, sobre el tema del botón y el doble click en el. Creo que va por el lado de los handlers de la page report-pet.

Hay puesto un onSubmit en el form y un onClick en el botón. Puede ser que entre ellos se pisen y prevenga las acciones del otro. Tal vez podrías delegar todo lo que está en el onClick al onSubmit que de todas formas va a dispararse al mismo tiempo. También, podrías usar el contenido del handleClick en un useEffect que levante un flag de un estado para que lo tome el handleSubmit y sepa a quien llamar. Con eso también te evitarías los otros flags.

Más a modo de recomendación, en el form tenés algunos tags que convendría evitar como sections y divs. En casos de form capaz es mejor dejar sólo fieldset, label, input, etc.

Saludos!!

RossiniMaximo commented 2 years ago

Buen dia Marcos!! Si estuve viendo a ver que podia ser , si es que era el problema de los botones interfiriendo y llegué a la raiz del problema (O eso creo). Cuestión que yo tengo como condición para reportar la mascota que el petName no sea un string vacío , entonces lo que estaría pasando es que ese string permanece vacío cuando haces el submit , lo loco es que le estoy seteando el name apenás se submitea pero por alguna razón diabolica no esta llegando , ya estoy tratando de arreglarlo con suerte lo saco rapido. Gracias x la revisión , si se te ocurre alguna sugerencia para el problema este del seteo es bienvenida, intente poniendole useEffect porque leí en internet algo pero no funciono , saludos!! :

El jue, 21 abr 2022 a las 19:19, Marcos Reuquén @.***>) escribió:

Buenas, Máximo! Me sumo al issue de Mati porque creo que tiene que ver.

Reporté una mascota y no me apareció en el inicio como que estaba cerca. Ahora, sobre el tema del botón y el doble click en el. Creo que va por el lado de los handlers de la page report-pet.

Hay puesto un onSubmit en el form y un onClick en el botón. Puede ser que entre ellos se pisen y prevenga las acciones del otro. Tal vez podrías delegar todo lo que está en el onClick al onSubmit que de todas formas va a dispararse al mismo tiempo. También, podrías usar el contenido del handleClick en un useEffect que levante un flag de un estado para que lo tome el handleSubmit y sepa a quien llamar. Con eso también te evitarías los otros flags.

Más a modo de recomendación, en el form tenés algunos tags que convendría evitar como sections y divs. En casos de form capaz es mejor dejar sólo fieldset, label, input, etc.

Saludos!!

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