Feedback

[EzequielVilla]

Buenas! Estoy viendo la pagina y te voy a pedir que arregles esto:

El espacio intermedio, dice que esta mostrando 3 resultados de 34 y solo hay 2 y que despues de varias veces de apretar "ver mas" empiezan a aparecer 3 items. Agregale el boton de "ver menos" asi el user puede volver.

• Por cada page dinamica (el item) me parece que no usaste getStaticProps, te lo recomiendo para que justamente cuando se ingrese esa data ya este cargada en el servidor y no se den estos pantallazos en los que no se muestra nada:

• Cuando quise hacer el login me sali antes de meter el codigo porque no me llegaba al mail (llego como a los 2 min, igual eso no importa tanto) pero la cuestion es que me logueo sin hacer nada, te dejo cap del local storage:

• La segunda vez que pedi la pass tambien tardo bastante, es raro que se demore.

• No me carga productos destacados:

• No me llego el email con la compra finalizada.

Respecto al codigo:

• Te recomiendo usar swr para el pagination asi si alguien busca dos veces lo mismo (o va a y viene con el pagination) no tiene que hacer el fetch de vuelta, sino que lo cachea.

• La capa pages tiene que ser mas limpia, tiene que cargar algo simple y llamar a los componentes que va a montar. Ahora que ya lo tenes todo ahi en pages te va a ser muy facil desglosarlo

const Home: NextPage = () => {
  return (
    <LayoutWithoutSearch>
      <HomeSearch />
      <OutstandingProducts></OutstandingProducts>
    </LayoutWithoutSearch>
  );
};

Mi home es asi por ej, y la idea es que cada page monte distintos componentes (tambien es mas facil de leer, cambiar, escalar), pero asi como lo tenes esta bueno para ya, como dije antes, ir separando en muchos.

• En pages/search no se si hace falta el useEffect(), estoy tratando de seguirle la logica y termina siendo redundante (para mi) las funciones pullProducts () y pullNewProductsPage(), es mas , dicen lo mismo.

Bueno, son varias cosas, hace todo esto tranqui y cualquier cosa consulta en el capitulo del discord!

[EzequielVilla]

Las pages cumplen la responsabilidad de un path (ruta) y no tanto de alojar logica. Por otra parte el componente Card creo que puede ir en UI porque es super reutilizable (como un boton)

[RossiniMaximo]

Hola Eze, muchas gracias por la review y por las recomendaciones , fueron de mucha ayuda la verdad :D. Estuve aplicando esos cambios estos días y hoy ya los deployee , posiblemente haya muchas otras cosas que pulir y si tenes alguna otra recomendación es bienvenida. En cuanto a los emails la verdad que no se que es lo que le pasa , es bastante frustrante porque hay veces en las que funciona de 10 y hay otras veces en los que tarda medio siglo en mandar el email , posiblemente lo que tenga que hacer es cambiar de herramienta para mandar emails , estoy usando sendGrid y hace rato me viene dando problema, además es un bondi meterse a la página y cuando te queres autenticar tener que hacer 10 mil malabares. Si tenes alguna recomendación de alguna herramienta para emails te lo agradecería , porque he estado buscando pero es como que nunca entiendo bien como funcan.Gracias y saludos Eze!!

El mar, 26 jul 2022 a las 16:38, Eze @.***>) escribió:

Las pages cumplen la responsabilidad de un path (ruta) y no tanto de alojar logica. Por otra parte el componente Card creo que puede ir en UI porque es super reutilizable (como un boton)

— Reply to this email directly, view it on GitHub https://github.com/RossiniMaximo/dwf-m10-challenge/issues/1#issuecomment-1195900738, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASD4RGKUU5JZKBIIUKLQIUDVWA5CRANCNFSM54XCKURA . You are receiving this because you are subscribed to this thread.Message ID: @.***>