@WP-API/amigos #reviewmerge

Related #888 and #862

@rmccue Thoughts?

Hmm, looks OK to me. With this change, we should still see [] back from an endpoint, but now get the query nav headers too, which is nice. What happens if you're way over the page limit though? (page=999 with only 2 posts in the collection)

What happens if you're way over the page limit though?

You'd still get query nav headers. I don't think that's unexpected

You'd still get query nav headers. I don't think that's unexpected

Right, but they'd link to ?page=998, which isn't where we should link IMO.

Here's what I'm thinking. We keep prev/next the same when you're on a page that "exists" (that is, the page is between 1 and max_num_pages). On all pages, link to rel=first and rel=last. This would ensure we never link to pages that don't exist, while still allowing navigation.

Seems reasonable. I'll update when I have a moment.

Will be handled in #1209