DenisMPar
commented
2 years ago Hola Ale! gracias por tu feedback, sirve mucho para mejorar! Definitivamente falta un refresh() en las mascotas cercanas. Voy a tener en cuenta todos esos detalles para ir puliéndolos a futuro.
Buenas Denis, felicitaciones por tu trabajo! Feedback:
Cuando reportas la mascota como encontrada, esta sigue apareciendo en mascotas cercanas. Lo mismo sucede cuando se reporta una mascota lejos de mi ubicación, esta no aparece en mascotas cercanas y está perfecto Pero si le cambio la ubicación al reporte de esa mascota por una ubicación cercana a la mía, esta no aparece en mis mascotas cercanas, hay que refrescar las page Quisas una posible solución sería usar el refresh(); de recoil
En mis datos estaría bueno también poder cambiar/modificar el mail
Las cards de mis mascotas perdidas se ven desproporcionada en algunas resoluciones, aplicándole un max-width: 330px; por ejem, se podría solucionar:
Poner cursor pointer a los botones, es algo sencillo que cambia mucho en la experiencia del usuario
Map al tener estado ya pasa a ser un component, los UI tienen que ser componentes super super atómicos, lo más chico del sistema, y si necesitan estado ya son un componente propio y Loader sería un UI, ya que no tiene estado y muy poca lógica.
Hay pages que están sobrecargadas de lógica, creo que la idea general es que no la tengan y solo llamen componentes, pero también como dijo Marce, son cuestiones que se van puliendo a medida que se va laburando con react y se analizan otros proyectos.
El funcionamiento form editar mascotas, está excelente
La app funciona sin problemas, el código está muy bien, los comentarios son un plus que siempre suma y se agradece, el desafío cumple con las condiciones aprobar, éxitos en lo que sigue!!