On 17 May 13:17, Finucane, Stephen wrote: > On 10 May 17:39, Andy Doan wrote: > > This adds the ability to expose a REST API based on the Django REST > > framework project. Since this project isn't packaged in most current > > distributions, we ensure that its both installed and enabled before > > trying to use it. > > > > Signed-off-by: Andy Doan <[email protected]> > > Inspired-by: Damien Lespiau <[email protected]> > > Reviewed-by: Stephen Finucane <[email protected]>
Actually, I do have one comment after playing around with this a little more: what's the possibility of using a 'Link' header instead of a payload envelope? Seems to be pretty easy to do [1], it's a proposed RFC standard [2], and it's consistent with the GitHub API (which I happen to admire) [3]. Certainly won't block on it though. Thoughts? Stephen [1] http://www.django-rest-framework.org/api-guide/pagination/#header-based-pagination [2] https://tools.ietf.org/html/rfc5988#section-5 [3] https://developer.github.com/guides/traversing-with-pagination/ _______________________________________________ Patchwork mailing list [email protected] https://lists.ozlabs.org/listinfo/patchwork
