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 pagination support to Event findAll() #17

Closed
Tschonti opened this issue May 24, 2024 · 0 comments · Fixed by #21
Closed

Add pagination support to Event findAll() #17

Tschonti opened this issue May 24, 2024 · 0 comments · Fixed by #21
Assignees
Labels
backend good first issue Good for newcomers

Comments

@Tschonti
Copy link
Member

Add query parameters to the findAll endpoint of the Event controller. If no parameters are passed, return all the events, but in a descending order based on the date.

Use offset based pagination in the service method, define the take and skip values as query parameters in the controller. Apply descending order based on the date in this case as well.

@Tschonti Tschonti added good first issue Good for newcomers backend labels May 24, 2024
@pem00 pem00 self-assigned this Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants