juliantrovato18 / pets-app

0 stars 0 forks source link

Feedback #1

Open EzequielVilla opened 2 years ago

EzequielVilla commented 2 years ago

Buenas Juli! Te voy a dejar lo que vi a simple vista:

image

image

Las opciones del burguer trata que salgan mas juntas y no asi:

image

Por otra parte cliqueo en cualquier parte y me va a "Mis datos" "Mascotas cerca tuyo", se ve que toma todo el box para hacerle click y no solamente el texto.

Los forms son del largo de la pagina :

image

Fijate por favor de probar mil y un veces el flujo de la pagina, proba todas las interacciones y que quede lindo que este es un tu primer laburo en react y es el que te va a quedar en tu portfolio por ahora! Te recomiendo probar el flujo creando usuarios nuevos y repitiendo todas las interacciones hasta que estes 100% que este bien (siempre un bug se puede encontrar pero que sea algo muy minimo).

No intente registrarme, asi que te dejo ese feedback (seguro las otras paginas tampoco quedan centradas en la version desktop)

Cualquier cosita consulta en el canal de discord o segui la charla aca!

juliantrovato18 commented 2 years ago

Hola eze, gracias por el feedback arregle todo lo que me dijiste, las interacciones tambien las veo bien y arregle esas cositas que me dijiste para que se vea mejor. lo vuelvo a mandar en un rato o mañana :D

EzequielVilla commented 2 years ago

Juli, no me puedo loguear y sigo viendo todo del lado izquierdo:

image

Cuando me quiero loguear queda asi:

image

Estaria bueno utilizar alguna animacion de que esta pasando algo (porque ahora no se si esto esta trabajando o no por ejemplo)

image

Y cuando voy a mis datos despues de intentar loguearme queda en blanco. Pero si no estoy logueado puedo modificar info? Es raro:

image

Cuando hago click en dar mi ubicacion no pasa nada:

image

Creo que la ui del desafio dice que si no encuentra nada que al menos diga que no hay nada cerca. Repito lo que dije antes, estaria bueno que cada boton de carga tenga algun minimo loading o algo , asi el user sabe que esta pasando algo, en este caso no se si esta buscando o si simplemente la palmo ahi.

Por alguna razon se imprime este token en mi consola:

image

Estaras seteando algun valor por default en el codigo? Por otra parte te dejo estos logs de cuando ingreso el email y la pass:

image

Te pido por favor que reveas el flujo, que intentes loguearte desde una ventana de incognito e intentes mil veces todas las interacciones. (reporta, reporta como encontrado, volve a reportar, editalo, eliminalo, acumula varios) La idea es que tengas este trabajo impecable para tu portfolio porque del front lo que importa mucho es el apartado visual ( y que el flujo tenga la menor cantidad de bugs, siempre uno se escapa de todas formas ).

EzequielVilla commented 2 years ago

image

Julii, te esta explotando el back, por eso no funciona bien. Fijate de correr en local el back (modulo7) y que tu local del M8 apunte a ese. Si todo funciona volve a levantar el m7 a produccion y que el local del M8 apunte a ese de produccion a ver que pasa, si revienta por alguna razon.

EzequielVilla commented 2 years ago

image

El ultimo cambio que hiciste puede estar afectando, dejaste que la api_base_url sea localhost

EzequielVilla commented 2 years ago

De todas formas, cuando levantes (es decir cuando no tengas errores en local en el m7 y en este) ,antes de entregarlo hace todo el flujo desde produccion y repeti hasta ver que no se rompa nada. Esto le habia pasado a otro de los chicos que despues de un ir y venir en la pagina se le caia y estuvimos dias hasta encontrar por que pasaba, pero lo entregaba, los tutores lo testeabamos y se le caia. Pero el test tiene que ser primero antes de entregarse, a lo sumo los tutores encuentrar fallas chiquitas o que se le escapan (que repito, siempre se te escapa algo chiquito, porque estas cansado o por otros motivos). Apoyate en discord, como siempre, y segui consultando cualquier cosa, estas cerca!

juliantrovato18 commented 2 years ago

hola eze, si obvio lo que menos quiero es joder a los tutores, yo creo que el problema era que en el back del 7 no habia hecho esos cambios porque el flujo lo probe y funcionaba, ahi arregle lo visual de esas dos pages que sinceramente se me paso, las demas estan centradas perfecctamente! y voy a buscar alguna animacion de carga para agregar ahora, muchas gracias por el feedback y perdon

EzequielVilla commented 2 years ago

No pasa nada Juli! Vos siempre pregunta cualquier duda! (tal vez sone medio brusco?) Fijate que onda ese modulo7 que suele traer inconvenientes aca jaja Cualquier cosa le podes meter try/catch para saber que es lo que esta explotando y asi evitas que se rompa todo el back.

Si queres una animacion rapida en mi repo de este modulo tengo un spinner medio a mano (es una fiaca buscar uno y adaptarlo)

FerDR89 commented 2 years ago

Hola Juli!

Algunas cosas que estuve viendo para modificar, sobre todo errores de CORS que no permite la comunicación con el BE.

Apenas abro la app ya me estalla la consola porque estas haciendo el fetch al BE consultando el endpoint de las mascotas cercanas.

Para que el user te de su ubicación dependes de un botón, lo que yo haría es asociar la llamada a la API a ese botón y que una vez que tenga la data ahí se dispare la llamada: Apreto el boton -> seteo la ubicación del user -> hago la llamada a la API

image

Luego de ingresar mi mail, lo estas mandando por la URL image

Cuando ingreso una pass me imprime la consola un xD image

Despues de un rato pude entrar en modo incognito: image

Sigue el mismo problema con el mapa que se te desplegaba sobre el menú (Lo solucionas poniéndole al modal un z-index en el Css)

Las mascotas no me las esta permitiendo reportar por los errores de CORS y lo mismo pasa si quisiera ver una ya reportada (En el desafío del mod dejaron unas líneas para agregar al BE para no tener problemas con esto) image

En el input de "tu nombre" si estoy logeado debería de aparecer mi nombre. En el modal debería de aparecer mi mail y la opción de cerrar sesión.

Cualquier cosa, escribime por DIscord y lo vamos viendo.

Abz y buen fin de semana

Alvax10 commented 2 years ago

Buenass @juliantrovato18 agregando a lo de @FerDR89 y @EzequielVilla, a mi se me rompe en la compu cuando pongo el modelo mobile:

Screenshot_20220711-213346_Gallery Screenshot_20220711-213343_Gallery Screenshot_20220711-213337_Gallery

Yo estaba en una compu que no era mia y me tiró un token en la consola(? Sea o no mío el token, no tendría que aparecer en la consola, tambien me quería loguear y me tiraba error de cors y se consologueaba un "xd"

Tratá de arreglar todos estos errores juli y ni lo mandes hasta que esté todo solucionado, ya que hay varios errores no solo de backend, sino también de frontend que es la parte más importante del desafío.

Acordate que podés apoyarte en discord para cualquier duda! 🙌🏻