MatiToledo / E-commerce-frontend

desafio-m10.vercel.app
1 stars 0 forks source link

Feedback #1

Open zapaiamarce opened 2 years ago

zapaiamarce commented 2 years ago

Mati, a nivel código está super. Hay cosas que vas a ir mejorando con la práctica y usando más y más herramientas. Te felicito! Con respecto a la web no pude usarla (te dejo captura) y te pido que le pegues una revisada general a toda la experiencia, sin sesión, con sesión, etc. Así probás que funcione bien todo lo básico.

image

Te dejo algunas cositas que podés mejorar en el código pero no son críticas para nada.

Me gustó como usaste los hooks, son complejos y vos los aplicaste bastante bien en general. También me copó como aplicaste SSR con getStaticProps(). Golazo! Muy buenos tus UI components! Guarda con darle muchos poderes (como en Card que hace un redirect) en estos casos mejor que reciba eso por props y que el ui componente sea lo menos inteligente posible. Va a escalar mejor. Muy bien usando los extends de styled, esta capa está impecable y es super importante. Parece que no, pero es la base para construir el resto. Pegale una mirada al error de la captura y a que todo funcione bien y devolvemelo que te lo apruebo de toque. Felicitaciones!!!

MatiToledo commented 2 years ago

Hola Marce!! No me habia dado cuenta del error para comprar sin haber iniciado sesión, hoy lo miro y lo reenvío al igual que con el resto de cosas. Muchas gracias por la devolución !