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.

Fix double slashes in HAL link urls - #1442

#1442
Closed
rachelbaker wants to merge 3 commits into
developWP-API/WP-API:developfrom
fix-link-urlsWP-API/WP-API:fix-link-urlsCopy head branch name to clipboard
Closed

Fix double slashes in HAL link urls#1442
rachelbaker wants to merge 3 commits into
developWP-API/WP-API:developfrom
fix-link-urlsWP-API/WP-API:fix-link-urlsCopy head branch name to clipboard

Conversation

@rachelbaker

Copy link
Copy Markdown
Member

HAL links and embedding are broken on sites with pretty permalinks as a result of #1426

@rachelbaker

Copy link
Copy Markdown
Member Author

@rmccue This fixes the _embed issue if pretty permalinks are enabled.
BUT if permalinks are set to the default structure, query parameters don't work. As an example: http://example.com/wp-json/wp/v2/posts?posts_per_page=4 returns a 301 with a Location Header of http://example.com/wp-json/wp/v2/posts/?posts_per_page=4

Any suggestions?

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.

2 participants

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