RossiniMaximo / dwf-m8-challengue-front

0 stars 0 forks source link

Nuevos issues #3

Open Alvax10 opened 2 years ago

Alvax10 commented 2 years ago

Buenas maxi, todo bien? estaba probando la página y hay algunos errores que se tienen que corregir (te los dividos en puntos):

1- Acordate que en este módulo lo que menos debe fallar es el front, y probando el menu con el responsive ocurrieron estas fallas: Podeés fijarte desde qe píxeles hasta que píxeles falla el front desde las imágenes :D

Captura de pantalla de 2022-04-22 10-35-05

Captura de pantalla de 2022-04-22 10-35-09

2- Tratando de reportar una mascota pasaron 2 cosas: La primera fue que e mapa no se veía (aunque tomaba las coordenadas igual) Y lo segundo fue que no me dejó reportar la mascota, me tiraba un error de cors, adjunto una imagen que muestra ambos errores en la consola:

Captura de pantalla de 2022-04-22 10-43-40

Como podés ver el mapa no se vé, y en la consola tira el error de por qué, y también podés ver en consola el error de cors al reportar la mascota.

Mis recomendaciones:

1- Para solucionar el problema del menu te diría que lo dejes en un lugar fijo, que no lo vayas moviendo dependiendo el tamaño, o de ultima, si la pantalla se hace muy grande, en vez de que haya un icono de menú podés hacer un menu-nav como se vió en módulos anteriores, te lo dejo a tu gusto, pero te recomendaría no complicartela tanto.

2- Estoy seguro de que el error del mapa es debido a mi wifi porque la token se ve que la está tomando, asique no toques nada de eso por las dudas.

3- Para revisar los problemas del back como el de reportar mascotas te recomiendo ir a los logs de heroku y ver en su consola si te tira un error más específico, ya que cuando aparece el error de cors por el back, la mayoría de veces es porque este se cae.

Te recuerdo que cualquier duda o complicación que tengas tennés el discord para apoyarte! Saludoss!

RossiniMaximo commented 2 years ago

Buenas Alvaro , gracias por la revisión , pude ser que este andando mal en producción, ahora estoy tratando de arreglar el problema del doble click del report , y creo que ya encontré el problema , no se me esta seteando el petName y yo tengo como condición para crear o updatear , de que el petName no este vacío digamos , el problema que estoy teniendo es que no se en que estoy fallando todavía , porque se lo estoy seteando enrealidad pero es como que no llega nunca ese seteo , apenás termino con esto me pongo con el problema de las resoluciones , gracias y saludos y si se te ocurre alguna sugerencia acerca de lo del petName es bienvenida!!

El vie, 22 abr 2022 a las 10:59, Alvaroalvaro @.***>) escribió:

Buenas maxi, todo bien? estaba probando la página y hay algunos errores que se tienen que corregir (te los dividos en puntos):

1- Acordate que en este módulo lo que menos debe fallar es el front, y probando el menu con el responsive ocurrieron estas fallas: Podeés fijarte desde qe píxeles hasta que píxeles falla el front desde las imágenes :D

[image: Captura de pantalla de 2022-04-22 10-35-05] https://user-images.githubusercontent.com/64299010/164728178-7b5c7451-344b-4e18-8bfe-f615a3c61091.png

[image: Captura de pantalla de 2022-04-22 10-35-09] https://user-images.githubusercontent.com/64299010/164728202-8d62448c-d81e-46bd-b3e9-dbacedea116a.png

2- Tratando de reportar una mascota pasaron 2 cosas: La primera fue que e mapa no se veía (aunque tomaba las coordenadas igual) Y lo segundo fue que no me dejó reportar la mascota, me tiraba un error de cors, adjunto una imagen que muestra ambos errores en la consola:

[image: Captura de pantalla de 2022-04-22 10-43-40] https://user-images.githubusercontent.com/64299010/164728423-71d69675-6e50-4bf5-9905-084ba82edb90.png

Como podés ver el mapa no se vé, y en la consola tira el error de por qué, y también podés ver en consola el error de cors al reportar la mascota.

Mis recomendaciones:

1- Para solucionar el problema del menu te diría que lo dejes en un lugar fijo, que no lo vayas moviendo dependiendo el tamaño, o de ultima, si la pantalla se hace muy grande, en vez de que haya un icono de menú podés hacer un menu-nav como se vió en módulos anteriores, te lo dejo a tu gusto, pero te recomendaría no complicartela tanto.

2- Estoy seguro de que el error del mapa es debido a mi wifi porque la token se ve que la está tomando, asique no toques nada de eso por las dudas.

3- Para revisar los problemas del back como el de reportar mascotas te recomiendo ir a los logs de heroku y ver en su consola si te tira un error más específico, ya que cuando aparece el error de cors por el back, la mayoría de veces es porque este se cae.

Te recuerdo que cualquier duda o complicación que tengas tennés el discord para apoyarte! Saludoss!

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

Alvax10 commented 2 years ago

Buenas maxi, dale! cualquier cosa si te quedás trabado acordate que en el canal de discord estamos para ayudarte!

Saludoss!

RossiniMaximo commented 2 years ago

Creo que ya pude solucionar lo del reporte , al menos en dev esta funcionando, igual es un avance yo pense que iba a ser imposible , ahora me fijo lo del aspecto de la página y otras cosas que capaz estan flojitas , muchas gracias Alvaro , cualquier cosa jodo por discord de ultima jejeje

El vie, 22 abr 2022 a las 11:33, Alvaroalvaro @.***>) escribió:

Buenas maxi, dale! cualquier cosa si te quedás trabado acordate que en el canal de discord estamos para ayudarte!

Saludoss!

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

Alvax10 commented 2 years ago

Creo que ya pude solucionar lo del reporte , al menos en dev esta funcionando, igual es un avance yo pense que iba a ser imposible , ahora me fijo lo del aspecto de la página y otras cosas que capaz estan flojitas , muchas gracias Alvaro , cualquier cosa jodo por discord de ultima jejeje El vie, 22 abr 2022 a las 11:33, Alvaroalvaro @.>) escribió: Buenas maxi, dale! cualquier cosa si te quedás trabado acordate que en el canal de discord estamos para ayudarte! Saludoss! — Reply to this email directly, view it on GitHub <#3 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASD4RGMC6ZUH3GYE4HMBIPTVGK2EJANCNFSM5UCNELHQ . You are receiving this because you commented.Message ID: @.>

Dalee tranqui maxi! saludoss

RossiniMaximo commented 2 years ago

Che Alvaro ahí lo estuve probando en incógnito de chrome y me funciona las busquedas con algolia y el report del pet en el primer click , si podes después lo probas y me decis como te fue xfa ? El report quizás tarda un segundito pero se manda al primer click , gracias :D

El vie, 22 abr 2022 a las 11:42, maximo rossini (< @.***>) escribió:

Creo que ya pude solucionar lo del reporte , al menos en dev esta funcionando, igual es un avance yo pense que iba a ser imposible , ahora me fijo lo del aspecto de la página y otras cosas que capaz estan flojitas , muchas gracias Alvaro , cualquier cosa jodo por discord de ultima jejeje

El vie, 22 abr 2022 a las 11:33, Alvaroalvaro @.***>) escribió:

Buenas maxi, dale! cualquier cosa si te quedás trabado acordate que en el canal de discord estamos para ayudarte!

Saludoss!

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

Alvax10 commented 2 years ago

dale maxi!

Alvax10 commented 2 years ago

Ahí lo probé y anduvo perfecto! te puedo dar el aprobado de mi parte, pero no se si leíste en el canal general de discord que solo los tutores van a poder aprobarte definitivamente y así poder pasar al siguiente módulo, pero por mi parte está aprobado!! suertee!

RossiniMaximo commented 2 years ago

jajaja dale Alvaro muchas gracias!! Ahi estoy dandole unos toquesitos a los botones y al fondo asi no es tan horrenda la page , saludos & suerte!!

El vie, 22 abr 2022 a las 13:11, Alvaroalvaro @.***>) escribió:

Ahí lo probé y anduvo perfecto! te puedo dar el aprobado de mi parte, pero no se si leíste en el canal general de discord que solo los tutores van a poder aprobarte definitivamente y así poder pasar al siguiente módulo, pero por mi parte está aprobado!! suertee!

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