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

Fazer o cache do user no localStorage #448

Merged
merged 5 commits into from
Jun 9, 2022
Merged

Conversation

filipedeschamps
Copy link
Owner

Branch intermediária da branch #443 que eu e @aprendendofelipe estamos fazendo para melhorar a experiência da interface decidir se o usuário está logado ou não.

Este é o deploy do Stage 1 da implementação, onde irá servir apenas para preencher o localStorage de quem já está logado. Mais detalhes da estratégia pode ser lido no PR original.

@vercel
Copy link

vercel bot commented Jun 9, 2022

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

Name Status Preview Updated
tabnews ✅ Ready (Inspect) Visit Preview Jun 9, 2022 at 5:13PM (UTC)

@filipedeschamps
Copy link
Owner Author

Aquele comportamento estranho da issue #445 tá até quebrando o job do Lint Styles 😂

image

E o medo de fazer merge disso e quebrar o buid lá na Vercel?

@aprendendofelipe
Copy link
Collaborator

E o medo de fazer merge disso e quebrar o buid lá na Vercel?

Quanto a isso, pode mandar ver que, se der erro no build, simplesmente irá continuar no ar a versão anterior

@filipedeschamps
Copy link
Owner Author

@aprendendofelipe em paralelo, eu fiz um commit que remove o user.email de ser salvo no localStoarge. E depois pensei, puts tem um monte de info ali sendo salva que não tem uso algum e difícil vai ter, então decidi salvar apenas o que é útil e não sensível: id, username, features. O que você acha?

@aprendendofelipe
Copy link
Collaborator

@aprendendofelipe em paralelo, eu fiz um commit que remove o user.email de ser salvo no localStoarge. E depois pensei, puts tem um monte de info ali sendo salva que não tem uso algum e difícil vai ter, então decidi salvar apenas o que é útil e não sensível: id, username, features. O que você acha?

Boa! Se não vai ser útil, não tem sentido guardar em cache. Como estava vindo tudo isso da API, eu pensei que tivesse algum plano de usar essas infos

@filipedeschamps
Copy link
Owner Author

Pinei a versão do Node para 16.15.0 e foi tudo 👍 vou ficar re-trying para ver se continua 100% e daí fazer merge na main 🤝

@filipedeschamps
Copy link
Owner Author

image

🤝 fazendo o merge na main 👍

@filipedeschamps filipedeschamps merged commit a3948be into main Jun 9, 2022
@filipedeschamps filipedeschamps deleted the cache-use-user2 branch June 9, 2022 17:21
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.

2 participants