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

Add support for cursor-based pagination #609

Merged
merged 8 commits into from
Jun 29, 2023
Merged

Conversation

paulrobertlloyd
Copy link
Collaborator

The Microsub specification uses cursor-based pagination. This has inspired a similar pagination extension for Micropub.

This PR replaces the previous page-based pagination with this cursor-based method. This solves a few things:

  • no longer need to provide a private _count value in Micropub responses
  • no longer need to address issue of long list of pages in pagination component
  • adheres to existing community standards and conventions

And of course this means one small part of the Microsub spec has already been built.

@paulrobertlloyd paulrobertlloyd added enhancement New feature or request documentation Good documentation benefits everyone plugin-endpoint Endpoint plug-in frontend labels Jun 29, 2023
@paulrobertlloyd paulrobertlloyd merged commit e3d975a into main Jun 29, 2023
@paulrobertlloyd paulrobertlloyd deleted the cursor-pagination branch June 29, 2023 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Good documentation benefits everyone enhancement New feature or request frontend plugin-endpoint Endpoint plug-in
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant