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

perf: small performance improvement #310

Merged
merged 1 commit into from
Dec 19, 2024
Merged

perf: small performance improvement #310

merged 1 commit into from
Dec 19, 2024

Conversation

PlayLikeNeverB4
Copy link
Contributor

Noticed this while working on another issue.

@davidgomes
Copy link
Collaborator

@PlayLikeNeverB4 this PR seems to include some things that are not relevant?

@PlayLikeNeverB4
Copy link
Contributor Author

Sorry about that, my local setup must have fallen out of sync somehow. Fixed it!

@duskpoet
Copy link
Member

Sorry about that, my local setup must have fallen out of sync somehow. Fixed it!

nah, I am pretty sure, it's mine force-push, sorry 😓

@@ -64,7 +64,7 @@ export const fillSingleProject = async (props: ProjectScopeProps) => {
if (props.projectId) {
return props;
}
const { data } = await props.apiClient.listProjects({});
const { data } = await props.apiClient.listProjects({ limit: 2 });
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🆒

@PlayLikeNeverB4 PlayLikeNeverB4 merged commit 2ac4e94 into main Dec 19, 2024
5 checks passed
@PlayLikeNeverB4 PlayLikeNeverB4 deleted the george/limit-2 branch December 19, 2024 12:28
github-merge-queue bot pushed a commit that referenced this pull request Dec 20, 2024
We don't need more than 2 projects in this response
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.

3 participants