Thanks everyone for the feedback. It generated this story which would switch the default to LimitOffsetPagination. Please raise any concerns you have with Pulp making this switch soon.
https://pulp.plan.io/issues/5324 On Wed, Aug 21, 2019 at 10:27 AM Justin Sherrill <jsher...@redhat.com> wrote: > > On 8/20/19 12:17 PM, Brian Bouterse wrote: > > Recently with pulp_ansible, users were interested in using pagination with > LimitOffsetPagination [0]. Pulp currently defaults to PageNumberPagination. > I looked at our current DRF defaults, and I noticed two things. > > 1. We default to the not-as-common PageNumberPagination based on examples > in the drf docs. > 2. We customize it here [1] in various ways. > > Can someone help me remember why these pagination style choices were made > or where the requirements came from? > Would our bindings work with a LimitOffsetPagination style? > What use cases drove the use and customization in this area? > > Also, @katello how would a pagination style change (like switching to > LimitOffsetPagination) affect you? > > Speaking for katello, we'd have to change a few lines of code, but it > seems like it would be minimal. > > > Thanks for any info you can provide. Maybe what we have right now is just > what we need, but I'm not sure. > > -Brian > > [0]: > https://www.django-rest-framework.org/api-guide/pagination/#setting-the-pagination-style > [1]: > https://github.com/pulp/pulpcore/blob/master/pulpcore/app/pagination.py > > _______________________________________________ > Pulp-dev mailing > listPulp-dev@redhat.comhttps://www.redhat.com/mailman/listinfo/pulp-dev > >
_______________________________________________ Pulp-dev mailing list Pulp-dev@redhat.com https://www.redhat.com/mailman/listinfo/pulp-dev