Skip to content
This repository was archived by the owner on Apr 27, 2026. It is now read-only.

feat: add pagination to listProjectSandboxes#90

Merged
odinuv merged 1 commit into
mainfrom
odin/AJDA-2441
Mar 19, 2026
Merged

feat: add pagination to listProjectSandboxes#90
odinuv merged 1 commit into
mainfrom
odin/AJDA-2441

Conversation

@odinuv
Copy link
Copy Markdown
Member

@odinuv odinuv commented Mar 19, 2026

https://linear.app/keboola/issue/AJDA-2441/sandboxes-service-run-migration-of-imageversion-size-shared

Description

Added pagination support to ManageClient::listProjectSandboxes(). The method now follows the Link response header to fetch all pages, extracting only the nextPageToken from it and reconstructing the correct URL to work around a known bug in sandboxes-api where the Link header points to the wrong endpoint.

Release Notes

Change Type
New feature — listProjectSandboxes now returns all sandboxes across all pages instead of only the first page.

Justification
Required to support the migration of imageVersion, size, and shared fields, which needs to enumerate all sandboxes for a project reliably.

Plans for Customer Communication
No customer communication needed — this is an internal API client change.

Impact Analysis
Low risk. The change is backwards-compatible; callers receive the same return type but now get a complete result set. No breaking changes or feature flags required.

Deployment Plan
Standard CI/CD continuous deployment across all stacks.

Rollback Plan
Standard git revert if issues detected post-deployment.

Post-Release Support Plan
No follow-up actions needed after release.

@linear
Copy link
Copy Markdown

linear Bot commented Mar 19, 2026

@odinuv odinuv requested a review from ErikZigo March 19, 2026 10:22
@odinuv
Copy link
Copy Markdown
Member Author

odinuv commented Mar 19, 2026

Copy link
Copy Markdown
Contributor

@ErikZigo ErikZigo left a comment

Choose a reason for hiding this comment

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

lgtm

@odinuv odinuv merged commit 84e3a72 into main Mar 19, 2026
1 check passed
@odinuv odinuv deleted the odin/AJDA-2441 branch March 19, 2026 14:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants