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

Listing activities with limit doesn't handle sorting correctly #10150

Closed
AlexAndBear opened this issue Sep 24, 2024 · 0 comments · Fixed by #10165
Closed

Listing activities with limit doesn't handle sorting correctly #10150

AlexAndBear opened this issue Sep 24, 2024 · 0 comments · Fixed by #10165
Assignees
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug

Comments

@AlexAndBear
Copy link
Contributor

Describe the bug

When listing activities with a limit,the limit is applied, but the sorting order is not correctly handled.

Steps to Reproduce:

  1. Have a list of 10 activities.
  2. Set a limit of 5 activities.
  3. Set the sort query to desc.

Expected Behavior:
The API should sort all activities and then return the last 5 activities when the sort query is set to desc.

Actual Behavior:
The API sorts the activities but returns the first 5 activities, ignoring the correct subset of activities that should be returned after sorting.

@AlexAndBear AlexAndBear added Type:Bug Priority:p2-high Escalation, on top of current planning, release blocker labels Sep 24, 2024
@AlexAndBear AlexAndBear changed the title Listing activities with limit doesn't take sorting into account Listing activities with limit doesn't handle sorting correctly Sep 24, 2024
@kulmann kulmann moved this from Qualification to Prio 2 in Infinite Scale Team Board Sep 25, 2024
@kobergj kobergj self-assigned this Sep 25, 2024
@kobergj kobergj moved this from Prio 2 to In progress in Infinite Scale Team Board Sep 25, 2024
@github-project-automation github-project-automation bot moved this from In progress to Done in Infinite Scale Team Board Sep 25, 2024
@micbar micbar mentioned this issue Oct 1, 2024
18 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants