Skip to content

Fix 404 links on API Platform's Philosophy #354

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

Merged
merged 2 commits into from
Dec 4, 2017

Conversation

marek-pietrzak-tg
Copy link
Contributor

Fixes links which currently lead to 404 pages.
Replaces Angular js link with the link to Client Generator docs.

Fixes links which currently lead to 404 pages. 
Replaces Angular js link with the link to Client Generator docs.
@@ -26,9 +26,9 @@ a full-stack all-in-one framework and a set of independent PHP components and bu
The architecture promoted by the framework will distrust many of you but read this tutorial until the end and you will
see how API Platform make modern development easy and fun again:

* [Start by **creating a hypermedia REST API**](distribution/index.md) exposing structured data that can
* [Start by **creating a hypermedia REST API**](/distribution/index.md) exposing structured data that can
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

../distribution/index.md would be better.

be understood by any compliant client such your apps but also as search engines (JSON-LD with Schema.org vocabulary).
This API is the central and unique entry point to access and modify data. It also encapsulates the whole business logic.
* [Then **create as many clients as you want using frontend technologies you love**](core/angularjs-integration.md): an HTML5/Javascript
* [Then **create as many clients as you want using frontend technologies you love**](/client-generator/index.md): an HTML5/Javascript
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same, ../client-generator/index.md would be better

@marek-pietrzak-tg
Copy link
Contributor Author

Changed to relative paths @sroze 👍

@sroze
Copy link
Contributor

sroze commented Dec 4, 2017

Thank you @mheki

@sroze sroze merged commit ded58e5 into api-platform:2.1 Dec 4, 2017
@marek-pietrzak-tg marek-pietrzak-tg deleted the patch-1 branch December 4, 2017 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants