Review

[alexismunoz1]

Buenas, Facu lo veo genial al desafio, los detalles estan excelentes

Me pasa que cuando quiero entrar a la page "Nueva mascota perdida" crashea la app

Arreglate eso y chiflame que lo miro

[facuariasla]

Hola Alexis, gracias! Si al parecer puse la variable de ambiente en cualquier lado, ahora anda en teoria, pero si deberia ver como ponerla en Firebase como env variable!

[alexismunoz1]

Buenas Facu, mil disculpas por la demora, ahí estuve chusmeando tu desafío y pienso que hay algunas cositas se podrían mejorar:

• Cuando querés reportar una mascota sin subir una imagen, no se puede hacer el reporte y eso está excelente, pero estaría bueno que al igual que con los demás campos como el nombre de mascota, te diga que es necesario subir una imagen de la mascota para poder hacer el reporte

• Cuando reporto una mascota sin indicar el market, por alguna razón se invierten las coordenadas, por ende se setea una ubicación incorrecta!

  

• A este texto estaría bueno hacerlo un poco más visible:

  

  Estaría bueno hacerlo más llamativo, ya que con tantos elementos en la interfaz, es usuario fácilmente puede pasar de ese texto sin saber de su existencia como me paso a mi jaja. Quisas con un: ⚠️important, al principio del texto puede ser más visible

• Se puede enviar un reporte del un avistamiento sin una descripción, estaría bueno que fuera un campo obligatorio, ya que sería el dato más importante del reporte.

• En queries/index.ts

  

  Declarar un valor de tipo "object" no es lo mejor, ya que se le pueda pasar cualquier cosa que no sea un valor primitivo, por ejemplo un array.

Este artículo lo explica muy bien: https://www.typescripttutorial.net/typescript-tutorial/typescript-object-type/

Como lo hiciste de esta manera es una excelente práctica:

• En cuanto a las pages, las veo muy cargadas de lógica y pienso que sería una buena práctica trasladar toda la lógica posible a componentes, o en el caso de que sean partes de código que se van a reutilizar y que no tengan estado, trasladarlo a uis. En parte también entiendo que lo hiciste de esa manera por usar chakra.

En fin, el desafío cumple con todas las condiciones para ser aprobado, creo que lo me das destaca este proyecto es la atención a los detalles que le pusiste, y las ganas de ir por algo más que el diseño original, esto sin dudas te puede hacer destacar en una prueba técnica por sobre otros candidatos, a seguir así y éxitos en lo que sigue!!!

[facuariasla]

• Excelente, casi todo lo que me comentas se me pasó por alto, por ejemplo lo de algún mensaje si falta la imágen, igual que lo de la inversión de coordenadas (se ve que es algo de lat lng intercambiados por error jaja) y las otras, así que miil gracias!
• Y respecto a lo del mal tipado y poca componentización concuerdo totalmente, es de lo primero que me di cuenta jaja, y es algo que tengo que mejorar a la hora de codear

Gracias Alexis me re sirven este tipo de feedback