<div dir="ltr">#3801 only describes the move from CursorPagination to PageNumberPagination. I don't think we considered using LimitOffsetPagination. <br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><br></div><div>David<br></div></div></div></div></div></div></div></div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 20, 2019 at 1:01 PM Dennis Kliban <<a href="mailto:dkliban@redhat.com">dkliban@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Aug 20, 2019 at 12:17 PM Brian Bouterse <<a href="mailto:bmbouter@redhat.com" target="_blank">bmbouter@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>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.</div><div><br></div><div>1. We default to the not-as-common PageNumberPagination based on examples in the drf docs.<br></div><div>2. We customize it here [1] in various ways.</div><div><br></div><div>Can someone help me remember why these pagination style choices were made or where the requirements came from?</div></div></blockquote><div><br></div><div>I believe the motivation is described here: <a href="https://pulp.plan.io/issues/3801" target="_blank">https://pulp.plan.io/issues/3801</a></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Would our bindings work with a LimitOffsetPagination style?</div></div></blockquote><div><br></div><div>Yes, the bindings will work with anything that uses query parameters for pagination.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>What use cases drove the use and customization in this area?<br></div><div><br></div><div>Also, @katello how would a pagination style change (like switching to LimitOffsetPagination) affect you?<br></div><div><br></div><div>Thanks for any info you can provide. Maybe what we have right now is just what we need, but I'm not sure.<br></div><div><br></div><div>-Brian<br></div><div><br></div><div>[0]: <a href="https://www.django-rest-framework.org/api-guide/pagination/#setting-the-pagination-style" target="_blank">https://www.django-rest-framework.org/api-guide/pagination/#setting-the-pagination-style</a></div><div>[1]: <a href="https://github.com/pulp/pulpcore/blob/master/pulpcore/app/pagination.py" target="_blank">https://github.com/pulp/pulpcore/blob/master/pulpcore/app/pagination.py</a></div></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div>