Skip to content
This repository has been archived by the owner. It is now read-only.

Use Compact links for embedded responses if they are available. #2412

Merged
merged 3 commits into from Mar 29, 2016

Conversation

@joehoyle
Copy link
Contributor

commented Mar 28, 2016

This fixes curies compatibility with 4.5, and doesn't fatal under 4.4

@joehoyle joehoyle added this to the 2.0 Beta 14 milestone Mar 28, 2016
@joehoyle joehoyle self-assigned this Mar 28, 2016
joehoyle added 2 commits Mar 28, 2016
@joehoyle

This comment has been minimized.

Copy link
Contributor Author

commented Mar 29, 2016

@WP-API/amigos #reviewmerge

@danielbachhuber

This comment has been minimized.

Copy link
Member

commented Mar 29, 2016

@joehoyle Why don't any of the tests need changing?

@joehoyle

This comment has been minimized.

Copy link
Contributor Author

commented Mar 29, 2016

@danielbachhuber all the tests are set up to use the full-uris, not compact URIs, we have a helper rest_extand_curies function or something similar.

@danielbachhuber

This comment has been minimized.

Copy link
Member

commented Mar 29, 2016

Ok.

@danielbachhuber danielbachhuber merged commit ea53f1c into develop Mar 29, 2016
3 checks passed
3 checks passed
Scrutinizer 1 new issues, 5 updated code elements
Details
codecov/project 94.46% (target 80.00%)
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@danielbachhuber danielbachhuber deleted the use-compact-links-when-available branch Mar 29, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
2 participants
You can’t perform that action at this time.