Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat: Add index of questions with mark in read #50

Closed
wants to merge 0 commits into from

Conversation

geekhadev
Copy link
Contributor

issue Close #47

  • Add endpoint for all questions
  • Add link in header to open question index screen
  • Add index screen
  • Refactor Card component for mark read

@vercel
Copy link

vercel bot commented Nov 11, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
preguntas-entrevista-react ✅ Ready (Inspect) Visit Preview Nov 11, 2022 at 4:27PM (UTC)

@geekhadev
Copy link
Contributor Author

@midudev si pudieras revisar mi PR porque traté de ver en Vercel porque no falló el Deploy pero no me da acceso a ver el log de error.

@midudev
Copy link
Owner

midudev commented Nov 11, 2022

El problema es que estás intentando acceder al localStorage en páginas desde el servidor. No puedes poner un acceso al localStorage en el cuerpo de un componente si se renderiza en el server (incluso aunque tengas el 'use client').

Deberías mover la lectura en el useEffect.

@geekhadev
Copy link
Contributor Author

Lo malo fue que quedaron dos indices de preguntas una en el home y la que hice en esta PR, voy a ver si creo un issue para unificarlas.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

List of questions read
2 participants