Skip to content

Remove dataset and episode offset pagination - #163

Merged
brookfxg merged 4 commits into
mainfrom
brook/dc-1789-migrate-python-dataset-and-episode-lists-off-offset
Sep 28, 2026
Merged

brookfxg merged 4 commits into
mainfrom
brook/dc-1789-migrate-python-dataset-and-episode-lists-off-offset

Conversation

@brookfxg

@brookfxg brookfxg commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • remove the offset argument from dataset and episode list APIs
  • remove legacy offset handling from the shared Page implementation
  • keep limit as the cursor page size and update pagination tests and documentation
  • set the package version to 0.21.0

Remove the offset argument and legacy pagination handling from the five dataset and episode list APIs in DC-1789. Keep limit as the cursor page size and update tests and documentation for cursor-only continuation.\n\nSet the package version to 0.20.1.
@brookfxg brookfxg added the python Pull requests that update python code label Sep 28, 2026
@linear-code

linear-code Bot commented Sep 28, 2026

Copy link
Copy Markdown

DC-1789

claude[bot]

This comment was marked as outdated.

claude[bot]

This comment was marked as outdated.

claude[bot]

This comment was marked as outdated.

@brookfxg
brookfxg marked this pull request as ready for review September 28, 2026 17:43
@brookfxg
brookfxg requested a review from mguida22 as a code owner September 28, 2026 17:43

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@brookfxg
brookfxg merged commit 732cde9 into main Sep 28, 2026
2 checks passed
@brookfxg
brookfxg deleted the brook/dc-1789-migrate-python-dataset-and-episode-lists-off-offset branch September 28, 2026 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

python Pull requests that update python code

Development

Successfully merging this pull request may close these issues.

2 participants