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

Lower time entry per_page to 5000 #244

Merged
merged 1 commit into from
Jan 24, 2024
Merged

Lower time entry per_page to 5000 #244

merged 1 commit into from
Jan 24, 2024

Conversation

b6b
Copy link
Contributor

@b6b b6b commented Jan 24, 2024

We are seeing some timeouts/500 error responses when fetching 10000 time entries per page. Fetching 5000 a page does not encounter the same issues.

@b6b b6b requested review from rickdav and ncperry January 24, 2024 00:19
@b6b b6b self-assigned this Jan 24, 2024
@b6b b6b added the reviewable label Jan 24, 2024
Copy link

@rickdav rickdav left a comment

Choose a reason for hiding this comment

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

🔥

@b6b b6b merged commit 7ba9696 into master Jan 24, 2024
2 checks passed
@b6b b6b deleted the lower-time-entry-page-size branch January 24, 2024 00:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants