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

Cache sources stats in Nuxt #5287

Closed
obulat opened this issue Dec 18, 2024 · 0 comments · Fixed by #5288
Closed

Cache sources stats in Nuxt #5287

obulat opened this issue Dec 18, 2024 · 0 comments · Fixed by #5288
Assignees
Labels
💻 aspect: code Concerns the software code in the repository ✨ goal: improvement Improvement to an existing user-facing feature 🟨 priority: medium Not blocking but should be addressed soon 🧱 stack: frontend Related to the Nuxt frontend

Comments

@obulat
Copy link
Contributor

obulat commented Dec 18, 2024

Problem

Currently, we send a /stats request to the API on every frontend incoming request. This is unnecessary and wasteful.

Description

Add a Nuxt server cache to only load the stats once an hour.

Alternatives

  • Save the data during the build time. This will only allow us to update it once a week during deploy, which is not often enough.

Additional context

@obulat obulat added ✨ goal: improvement Improvement to an existing user-facing feature 💻 aspect: code Concerns the software code in the repository 🟨 priority: medium Not blocking but should be addressed soon 🧱 stack: frontend Related to the Nuxt frontend labels Dec 18, 2024
@obulat obulat self-assigned this Dec 18, 2024
@openverse-bot openverse-bot moved this to 📋 Backlog in Openverse Backlog Dec 18, 2024
@openverse-bot openverse-bot moved this from 📋 Backlog to 🏗 In Progress in Openverse Backlog Dec 18, 2024
@openverse-bot openverse-bot moved this from 🏗 In Progress to ✅ Done in Openverse Backlog Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💻 aspect: code Concerns the software code in the repository ✨ goal: improvement Improvement to an existing user-facing feature 🟨 priority: medium Not blocking but should be addressed soon 🧱 stack: frontend Related to the Nuxt frontend
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant