Skip to content

Add headers pagination mixin for generic api view or viewsets. #1168

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

niwinz
Copy link
Contributor

@niwinz niwinz commented Oct 10, 2013

This implements a simple way to remove a pagination metadata from response body. It helps to use a list json response as is, in javascript frameworks (without previous adaptation process).

I don't know if location of mixin is correct. Also, I don't know if it should be on rest-framework. But if you consider it useful for include into rest-framework, I will add doc and tests.

This pull-request is slightly similar to #1167 but implement a more generalist way to put metadata on headers instead of populate a body. In my opinion, put Link header with all data encoded as html/xml is making dificult to parse than simple value.

@jespino
Copy link
Contributor

jespino commented Oct 10, 2013

+1

@kevin-brown
Copy link
Member

See the current discussion in #1167 on pagination.

@tomchristie
Copy link
Member

Closing in favor of #1169. See this comment #1167 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants