Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Sep 24, 2018. It is now read-only.

filter offset enable - #1471

#1471
Closed
tigusigalpa wants to merge 1 commit into
WP-API:developWP-API/WP-API:developfrom
tigusigalpa:filter-offset-enableCopy head branch name to clipboard
Closed

filter offset enable#1471
tigusigalpa wants to merge 1 commit into
WP-API:developWP-API/WP-API:developfrom
tigusigalpa:filter-offset-enableCopy head branch name to clipboard

Conversation

@tigusigalpa

Copy link
Copy Markdown

No description provided.

@rmccue

rmccue commented Aug 10, 2015

Copy link
Copy Markdown
Member

You should be able to handle this already through the page parameter (that is ?page=2). Is there a reason you need offset as well?

@tigusigalpa

Copy link
Copy Markdown
Author

Hi, Ryan! I think we need the one because if we need to get all posts with some offset or posts_per_page the page param is very poor for it

@jcutrell

Copy link
Copy Markdown

I'd like to do a 👍 on this - there are many cases where offset is needed and page/posts_per_page doesn't quite cut it. Is there a particular reason this parameter needs to be hidden behind authentication?

@danielbachhuber

Copy link
Copy Markdown
Member

Closing in favor of #924. We should decide whether we want to support offset as a global parameter for collections. You can add support in your local instance of WordPress by filtering query_vars

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.