Skip to content
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

Updated "Learn more from the Cookbook" section #6222

Closed
wants to merge 2 commits into from

Conversation

sfdumi
Copy link
Contributor

@sfdumi sfdumi commented Feb 3, 2016

Added missing links from the cookbook.

Added missing links from the cookbook.
* :doc:`/current/cookbook/routing/service_container_parameters`
* :doc:`/current/cookbook/routing/custom_route_loader`
* :doc:`/current/cookbook/routing/redirect_trailing_slash`
* :doc:`/current/cookbook/routing/extra_information`
Copy link
Member

Choose a reason for hiding this comment

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

You have to remove the /current prefix here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks. Missed that.

Removed `/current` prefix
@xabbuh
Copy link
Member

xabbuh commented Feb 8, 2016

👍

@wouterj
Copy link
Member

wouterj commented Feb 9, 2016

Thank you for updating this section @sfdumi! Seems like we haven't updated this list for a long time. I've merged your PR in the 2.3 branch of the docs and will merge it into the newer versions from there (this is the reason behind the closed status instead of the merged one).

wouterj added a commit that referenced this pull request Feb 9, 2016
This PR was submitted for the 3.0 branch but it was merged into the 2.3 branch instead (closes #6222).

Discussion
----------

Updated "Learn more from the Cookbook" section

Added missing links from the cookbook.

Commits
-------

80ac8df Updated "Learn more from the Cookbook" section
@wouterj wouterj closed this Feb 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants