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.

Sign up for GitHub

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

Jump to bottom

[CAL-5212] Don't fetch all data if only one possible lucky user #19503

Open
CarinaWolli opened this issue Feb 24, 2025 · 0 comments
Open

[CAL-5212] Don't fetch all data if only one possible lucky user #19503

CarinaWolli opened this issue Feb 24, 2025 · 0 comments
Assignees
@emrysal
Labels
High priority Created by Linear-GitHub Sync performance area: performance, page load, slow, slow endpoints, loading screen, unresponsive

Comments

@CarinaWolli
Copy link
Member

There is no need to fetch all data if there is only one available user. We return early in getLuckyUser_requiresDataToBePreFetched but we should already return early before we fetch the data in getLuckyUser

From SyncLinear.com | CAL-5212
@CarinaWolli CarinaWolli added the High priority Created by Linear-GitHub Sync label Feb 24, 2025
@dosubot dosubot bot added the performance area: performance, page load, slow, slow endpoints, loading screen, unresponsive label Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@emrysal emrysal

Labels
High priority Created by Linear-GitHub Sync performance area: performance, page load, slow, slow endpoints, loading screen, unresponsive
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@emrysal @CarinaWolli