MrJoustinOF / dwf-m8-desafio

0 stars 0 forks source link

Feedback #2

Open EzequielVilla opened 2 years ago

EzequielVilla commented 2 years ago

Buenas! La pagina esta muy buena, me gusta el manejo de errores que tenes al ingresar datos en los forms! Te dejo un par comentarios respecto al codigo nomas, pero segun tengo entendido es cuestion de practica y de ver otros trabajos con react :

-El llamado a la api desde los componentes mismos, eso centralizalo en un modulo que tenga esa misma logica, asi si algun dia se modifica la api o el programa no funciona, sabes a donde ir y no estas buscando archivo por archivo a ver cual lo maneja.

-Tenes un customHook usado en un componente (MeForm por ejemplo), podes usarlo como usaste los otros y lo mismo del punto anterior, centralizar esa logica en un lugar.

-Los ui de button por ejemplo estan muy bien, son cortitos y modifican el elemento para reutilizarse, te dejo un comentario de Marce que hizo en otro review : "como regla general los ui components deberían ser atomos, los componentes más chicos de tu sistema: botones, textfields etc. Si ya necesita un estado, es un componente hecho y derecho (capa components) esto te va a dejar elegir/buscar mejor el lugar de componentes cuando tengas muchos más." Lo explica bastante bien y me parece que se aplica en este caso.

Buen laburo che!