Skip to content

Instantly share code, notes, and snippets.

@coliveravenegas
Created December 7, 2021 19:03
Show Gist options
  • Save coliveravenegas/8226bf357cdbda68bd06a1db851463b2 to your computer and use it in GitHub Desktop.
Save coliveravenegas/8226bf357cdbda68bd06a1db851463b2 to your computer and use it in GitHub Desktop.
Resumen
- React: (2/5 stars)
- Usa custom hooks para separar logica como useCountries
- Conoce existencia de hooks de optimización como useCallback y useMemo pero los aplica necesariamente bien, pero es un buen comienzo.
- Condiciona componentes y props que a veces pueden dificultar la lectura del código.
- Agrega complejidad innecesaria al código (podría ser que quiera impresionar y en realidad en la vida real no lo hace, pero de no ser así, es una mala practica)
- La aplicación genera re-renders innecesarios pero no estamos calificando el tema de performance exactamente, a pesar de que el candidato intenta usar hooks qué mejoran el performance, no esta logrando mejorar el performance de la app al final.
- Usa incorrectamente react router para ir al Home del app, podría utilizar un Link pero usa un div, semánticamente incorrecto
- En materia de react router no hay mucho que calificar, usa lo necesario para el project
- Podia aprovechar para optimizar el Search input para evitar hacer búsqueda en cada word types usando algún tipo de debounce o similar.
- Typescript: (2/5 stars)
- Usa interfaces en lugar de types pero no hace uso de características especificas de los interfaces con lo que pudo usar types tranquilamente.
- Usa innecesariamente uno que otro utility type, el ejercicio tampoco es que demande un uso intermedio o avanzado de utility types. Se ve un uso mas interesante de types como props entre interfaces para tapar responses de Apollo

- Graphql (1/5 stars)
- Conocimiento basico para configurar apollo y ejecutar queries, nada adicional aquí, la prueba tampoco lo demanda
- No usa loaders para .graphql
- No usa fragments
- Other
- No usa lib de css, que indique alguna habilidad con ello
- Estructura el proyecto de manera personal o al menos no parece seguir algún tipo de estructura conocida, hay una mezcla de folders que contiene api, queries de graphql
- No usa tests de ningún tipo, que podrían demostrar un poco mas de nivel como Dev.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment