Skip to content
This repository was archived by the owner on Sep 24, 2018. It is now read-only.

Fix double slashes in HAL link urls #1442

Closed
rachelbaker wants to merge 3 commits into from
Closed

Fix double slashes in HAL link urls #1442

rachelbaker wants to merge 3 commits into from

Conversation

@rachelbaker
Copy link
Member

@rachelbaker rachelbaker commented Jul 29, 2015

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

@rachelbaker
Copy link
Member Author

rachelbaker commented Jul 30, 2015

@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?

@rmccue rmccue mentioned this pull request Aug 3, 2015
Merged
@rmccue rmccue closed this Aug 7, 2015
@rmccue rmccue deleted the fix-link-urls branch August 7, 2015 19:48
@dave-green-uk dave-green-uk mentioned this pull request Jun 25, 2016
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rachelbaker @rmccue