rafalynch / lost-pets-react

Proyecto para apx - Modulo 8 - React
0 stars 0 forks source link

Feedback #1

Open EzequielVilla opened 2 years ago

EzequielVilla commented 2 years ago

Buenas! Estoy teniendo un problema al querer reportar, no me deja, me dice que llene todos los campos y ya hice todo, te dejo una captura de como lo tengo

image

rafalynch commented 2 years ago

Buenas! Estoy teniendo un problema al querer reportar, no me deja, me dice que llene todos los campos y ya hice todo, te dejo una captura de como lo tengo

image

Hola! Creo que es un tema de la búsqueda en el mapa. No se si es un problema de mapbox, pero a veces hay que apretar "enter" un par de veces hasta que funcione.

EzequielVilla commented 2 years ago

Joyaaa, ahi pude!. Lo unico que note es que cuando haces click en "reportar como encontrado" sigue apareciendo en el home como mascota perdida cerca, capaz hay que sacarlo o ponerle "encontrado". Si pongo despublicar si desaparece, eso nomas, ahora chequeo el codigo!

rafalynch commented 2 years ago

Que raro, a mi eso no me pasa. Avisame si ves cual es el problema. Gracias!

EzequielVilla commented 2 years ago

Listo, ahi no me aparece en el home, que raro. Bueno, respecto al codigo, me falta seguir mirandolo, pero hasta donde llegue a ver esta bien. Las pages tienen un poco de logica pero nada del otro mundo (creo que la idea es llegar a logica 0, pero Marce dijo que las practicas de react se mejoran mirando ejemplos y haciendo muchos laburos), las UI estan perfectas, centralizaste bien los llamados de la api en el archivo api.ts. Por mi esta aprobado eh! Me falta mirarlo un poco mas pero esta bien.

rafalynch commented 2 years ago

Dale, muchas gracias!