Project

Profile

Help

Issue #5489

closed

NamePagination mimics PageNumberPagination while it should inherit LimitOffsetPagination

Added by mdellweg over 4 years ago. Updated almost 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
API Bindings, Easy Fix
Sprint:
Sprint 60
Quarter:

Description

This leads to inconsistent pagination apis.

See:
https://pulp.plan.io/issues/5324

Maybe it is even possible to inherit from DEFAULT_PAGINATION_CLASS.

Actions #1

Updated by mdellweg over 4 years ago

It might be even easier to use OrderingFilter as described here:
https://www.django-rest-framework.org/api-guide/pagination/#configuration_2

Actions #2

Updated by mdellweg over 4 years ago

In case, this is going to be a thing, i have started a PR:

https://github.com/pulp/pulpcore/pull/308

Actions #3

Updated by fao89 over 4 years ago

  • Triaged changed from No to Yes
  • Sprint set to Sprint 59
Actions #4

Updated by mdellweg over 4 years ago

  • Status changed from NEW to POST
Actions #5

Updated by rchan over 4 years ago

  • Sprint changed from Sprint 59 to Sprint 60

Added by mdellweg over 4 years ago

Revision 85407f15 | View on GitHub

Remove NamePagination class

Sorting of the results can be accomplished on the queryset.

fixes #5489 https://pulp.plan.io/issues/5489

Actions #6

Updated by mdellweg over 4 years ago

  • Status changed from POST to MODIFIED
Actions #7

Updated by bmbouter over 4 years ago

  • Sprint/Milestone set to 3.0.0
Actions #8

Updated by bmbouter over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions #9

Updated by bmbouter almost 4 years ago

  • Category deleted (14)

We are removing the 'API' category per open floor discussion June 16, 2020.

Also available in: Atom PDF