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 intodevelopfrom
fix-link-urls
Closed

Fix double slashes in HAL link urls#1442
rachelbaker wants to merge 3 commits intodevelopfrom
fix-link-urls

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.

Labels

None yet

2 participants

@rachelbaker @rmccue