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

Source Zendesk Support: ticket_metrics stream pagination limit #26055

Closed
1 task
williamangelo opened this issue May 14, 2023 · 3 comments
Closed
1 task

Source Zendesk Support: ticket_metrics stream pagination limit #26055

williamangelo opened this issue May 14, 2023 · 3 comments
Labels

Comments

@williamangelo
Copy link

williamangelo commented May 14, 2023

Connector Name

source-zendesk-support

Connector Version

0.2.29

What step the error happened?

During the sync

Revelant information

TicketMetrics stream is defined as cursor pagination (#13757) in order to avoid rate limits of the Zendesk API. This behaviour is currently not working as intended, the api calls are still paginated via offset mode.

Syncs that fetch more than 100k records will still reach pagination limits and return Response Text: Pagination limit reached for offset model. According to the Zendesk documentation on cursor pagination the correct parameter for page size is page[size] (instead of the current per_page), omitting this parameter will default to offset pagination.

Relevant log output

No response

Contribute

  • Yes, I want to contribute
@williamangelo williamangelo added area/connectors Connector related issues needs-triage type/bug Something isn't working labels May 14, 2023
@williamangelo williamangelo changed the title Zendesk Support ticket_metrics stream pagination limit Source Zendesk Support: ticket_metrics stream pagination limit May 15, 2023
@lideke
Copy link
Contributor

lideke commented May 31, 2023

same issue : #26748

@btkcodedev
Copy link
Collaborator

I would like to work on this

@btkcodedev
Copy link
Collaborator

Already been solved through #26945
Check comments in #27070 for reference
Please close issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
5 participants