New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change link relations to use api.w.org #1726

Merged
merged 1 commit into from Nov 17, 2015

Conversation

Projects
None yet
3 participants
@danielbachhuber
Member

danielbachhuber commented Nov 10, 2015

Fixes #1710

@danielbachhuber danielbachhuber added this to the 2.0 Beta 6 milestone Nov 10, 2015

@danielbachhuber

This comment has been minimized.

Show comment
Hide comment
@danielbachhuber
Member

danielbachhuber commented Nov 10, 2015

@WP-API/amigos #reviewmerge

@danielbachhuber

This comment has been minimized.

Show comment
Hide comment
@danielbachhuber
Member

danielbachhuber commented Nov 17, 2015

danielbachhuber added a commit that referenced this pull request Nov 17, 2015

Merge pull request #1726 from WP-API/1710-link-relations
Change link relations to use api.w.org

@danielbachhuber danielbachhuber merged commit ba185d1 into develop Nov 17, 2015

4 checks passed

Scrutinizer No new issues
Details
codecov/project 92.89% (target 80.00%)
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@danielbachhuber danielbachhuber deleted the 1710-link-relations branch Nov 17, 2015

@danielbachhuber danielbachhuber modified the milestones: 2.0 Beta 8, 2.0 Beta 6.1 Nov 18, 2015

@robertharm

This comment has been minimized.

Show comment
Hide comment
@robertharm

robertharm Nov 19, 2015

just wondering if the links should use https instead of http, according to https://core.trac.wordpress.org/changeset/35650 ?

robertharm commented on e4a9fec Nov 19, 2015

just wondering if the links should use https instead of http, according to https://core.trac.wordpress.org/changeset/35650 ?

This comment has been minimized.

Show comment
Hide comment
@rmccue

rmccue Nov 19, 2015

Member

@robertharm Good catch, these should definitely all be HTTPS

Member

rmccue replied Nov 19, 2015

@robertharm Good catch, these should definitely all be HTTPS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment