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

[Umami] Remove a busca de dados estatísticos de cada conteúdo durante a geração das páginas estáticas #1834

Merged
merged 1 commit into from
Dec 30, 2024

Conversation

aprendendofelipe
Copy link
Collaborator

Com isso, encerra o experimento sobre a Umami Cloud iniciado no PR #1832.

Considero que obtivemos sucesso ao conseguir as seguintes respostas:

  1. Podemos substituir a Vercel Analytics, pois a Umami nos traz mais dados e funcionalidades com um menor custo. ✅
  2. Os limites da API deram conta da nossa demanda normal. Tivemos raros erros com visitas de Bots. ✅
  3. A performance da API impede que os dados sejam buscados durante a geração das páginas estáticas ❌

Sobre a performance, vejam o impacto na duração da geração das páginas de conteúdos após o merge do #1832. O tempo médio subiu da casa dos 250 ms para cerca de 2 000 ms:

image

Conclusão

Por causa da performance, não é viável buscar as estatísticas na getStaticProps, mas podemos perfeitamente fazer a busca pelo client usando um endpoint com cache. E a busca pelo client deve reduzir ainda mais os raríssimos erros que tivemos com a API.

Esse PR ainda não implementa isso, pois precisamos de uma decisão final sobre a mudança da Vercel para a Umami, mas acredito que essa seja uma mudança bem vinda.

Copy link

vercel bot commented Dec 30, 2024

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

Name Status Preview Comments Updated (UTC)
tabnews ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 30, 2024 1:14pm

@aprendendofelipe aprendendofelipe merged commit a660b22 into main Dec 30, 2024
7 checks passed
@aprendendofelipe aprendendofelipe deleted the remove-content-statistics branch December 30, 2024 13:18
@filipedeschamps
Copy link
Owner

Esse PR ainda não implementa isso, pois precisamos de uma decisão final sobre a mudança da Vercel para a Umami, mas acredito que essa seja uma mudança bem vinda.

Mudaria total 🤝

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