<div dir="ltr"><div>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.</div><div><br></div><div><a href="https://pulp.plan.io/issues/5324">https://pulp.plan.io/issues/5324</a></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Aug 21, 2019 at 10:27 AM Justin Sherrill <<a href="mailto:jsherril@redhat.com">jsherril@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 bgcolor="#FFFFFF">
    <p><br>
    </p>
    <div class="gmail-m_-3347141412564007341moz-cite-prefix">On 8/20/19 12:17 PM, Brian Bouterse
      wrote:<br>
    </div>
    <blockquote type="cite">
      
      <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>Would our bindings work with a LimitOffsetPagination style?</div>
        <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>
    </blockquote>
    <p>Speaking for katello, we'd have to change a few lines of code,
      but it seems like it would be minimal.</p>
    <blockquote type="cite">
      <div dir="ltr">
        <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>
      <fieldset class="gmail-m_-3347141412564007341mimeAttachmentHeader"></fieldset>
      <pre class="gmail-m_-3347141412564007341moz-quote-pre">_______________________________________________
Pulp-dev mailing list
<a class="gmail-m_-3347141412564007341moz-txt-link-abbreviated" href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a>
<a class="gmail-m_-3347141412564007341moz-txt-link-freetext" href="https://www.redhat.com/mailman/listinfo/pulp-dev" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a>
</pre>
    </blockquote>
  </div>

</blockquote></div>