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

Remove talk of cursor based pagination model #7989

Closed
2 tasks
gordsport opened this issue Mar 27, 2023 · 0 comments · Fixed by #8086
Closed
2 tasks

Remove talk of cursor based pagination model #7989

gordsport opened this issue Mar 27, 2023 · 0 comments · Fixed by #8086

Comments

@gordsport
Copy link
Contributor

Issue

In a lot, but of course not all, the APIs that list things we say “using a cursor based pagination model”.

This is NOT necessary because it can be inferred from the request/response fields.

Additionally, in this case, we have cursor based AND offset/limit so its not factually correct!

Tasks

  • Look for instances where we say "using cursor based...." and see if this is inferred and remove if so
  • Look for instances where we state cursor based and if its not factually correct tidy up or remove.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant