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.

Sign up for GitHub

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

Jump to bottom

Ability to add cursor pagination #735

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Ability to add cursor pagination #735

wants to merge 1 commit into from

Conversation

fikri-kompanion
Copy link

Extend model pagination with cursor pagination.

Example:

@apiResourceModel App\Models\User paginate=10,cursor

@fikri-kompanion
Ability to add cursor pagination
9950958 
Extend model pagination with cursor pagination.

```
@apiResourceModel App\Models\User paginate=10,cursor
```
@shalvah
Copy link
Contributor

shalvah commented Sep 29, 2023

You need to add tests 😬

@shalvah
Copy link
Contributor

shalvah commented Jun 17, 2024

Hey, are you still interested in adding this? Do you need help adding tests?

@fikri-kompanion
Copy link
Author

@shalvah that would be good

@olivernybroe
Copy link

Hey 👋 Was about to open a PR for this but saw this one already existed.

Is this currently being worked on? Else I don't mind giving a hand with the missing tests.
Also the ResponseFromApiResource attribute should prob. also get a property for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
awaiting info
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fikri-kompanion @shalvah @olivernybroe