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

fix: correct limits for dataset get items endpoint #1393

Merged
merged 1 commit into from
Jan 9, 2025

Conversation

metalwarrior665
Copy link
Member

See test runs:
2000 columns - https://console.apify.com/view/runs/zQPSs9IcesvwlZfiw
unlimited items - https://console.apify.com/view/runs/mnPtQvpRJCLtFevg3

I think these changes are a few years old by now so it would be good if someone took a good look.

Copy link
Member

@B4nan B4nan left a comment

Choose a reason for hiding this comment

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

fine by me, but let's also include @drobnikj or someone else from the platform team to confirm the changes

@B4nan B4nan requested a review from drobnikj January 9, 2025 07:46
Copy link
Member

@drobnikj drobnikj left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@metalwarrior665 metalwarrior665 merged commit bbfe824 into master Jan 9, 2025
8 checks passed
@metalwarrior665 metalwarrior665 deleted the fix/dataset-items-limits branch January 9, 2025 20:35
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.

None yet

3 participants