Skip to content

Instantly share code, notes, and snippets.

@heanfig

heanfig/CRISTIAN.md

Created July 18, 2022 23:35
Show Gist options
  • Save heanfig/ee1eec46234e3fbf94167a699970d39a to your computer and use it in GitHub Desktop.
Save heanfig/ee1eec46234e3fbf94167a699970d39a to your computer and use it in GitHub Desktop.
Download ZIP
Cristian
Raw
CRISTIAN.md

Calificación 7.5 / 10: Se evidencia que trabajó en la prueba y que la prueba tecnica es completamente funcional e impelemnto estructuras HTML y de JSX

Observaciones Positivas

  • Uso tailwind como dependencia
  • Interfaz intuitiva
  • Posiblemente todos los casos de uso quedaron funcionales
  • Implemento getServerSideProps y getStaticPaths
  • Hizo uso de efectos
  • Lo desplegó en vercel
  • Implementó los hooks de GraphQL
  • Hizo uso de interfaces
  • Lo desplegó en vercel

    • Observaciones Negativas

  • No uso completamente todas las caracteristicas de tailwind (tailwind.config.js vacío)
  • Mal uso de la extensión .tsx (apolloClient.tsx)
  • No creó un document.tsx o app.tsx para evitar repetir ApolloProvider y la estructura HTML en todos los componentes
  • Es una mala practica poner todos tipos en types.d.ts ya que es un archivo de definicion
  • Los tipos deben usar su tipo de dato primitivo no su clase
    • Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment