Skip to content

fix(mod): paginate getApps using scanned cursor, not count#135

Merged
charlesHetterich merged 1 commit intomainfrom
rh/fix-pagination
May 7, 2026
Merged

fix(mod): paginate getApps using scanned cursor, not count#135
charlesHetterich merged 1 commit intomainfrom
rh/fix-pagination

Conversation

@ReinhardHatko
Copy link
Copy Markdown
Collaborator

Fixes duplicate entries in dot mod list.

Closes #134

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Dev build ready — try this branch:

curl -fsSL https://raw.githubusercontent.com/paritytech/playground-cli/main/install.sh | VERSION=dev/rh/fix-pagination bash

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

E2E Test Pass · ✅ PASS

Tag: e2e-ci-pr · Branch: rh/fix-pagination · Commit: f02124e · Run logs

Cell Result Time
pr-install ✅ PASS 0m44s
pr-deploy-foundry ✅ PASS 2m17s
pr-preflight ✅ PASS 3m50s
pr-init-session ✅ PASS 1m49s
pr-deploy-frontend ✅ PASS 8m47s
pr-mod ✅ PASS 1m25s
pr-deploy-cdm ✅ PASS 5m24s
${{ matrix.cell }} ⏭️ SKIP 0m-1s
${{ matrix.cell }} ⏭️ SKIP 0m-1s

Sentry traces: view spans for this run

Copy link
Copy Markdown
Collaborator

@charlesHetterich charlesHetterich left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@charlesHetterich charlesHetterich merged commit feff1e5 into main May 7, 2026
19 checks passed
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.

dot mod: app browser uses wrong cursor when paginating getApps

2 participants