Skip to content

Update routing.rst. Explain using url() v. path(). #3732

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

Closed
wants to merge 1 commit into from

Conversation

ackerman
Copy link
Contributor

The original documentation seemed to imply that, when generating absolute URLs from a template, the argument TRUE should be supplied to the url() method. This change explains that in Twig, one uses the url() method to generate an absolute URL.

@wouterj
Copy link
Member

wouterj commented Mar 28, 2014

👍 I like it!

@wouterj
Copy link
Member

wouterj commented Mar 28, 2014

btw, @weaverryan this should be merged into 2.3. (@ackerman, it's not your fault this has gone wrong, it's an unfixable bug with the edit button)

@weaverryan
Copy link
Member

Yes, I like it much better too! Thanks @ackerman!

weaverryan added a commit that referenced this pull request Apr 2, 2014
…man)

This PR was submitted for the 2.4 branch but it was merged into the 2.3 branch instead (closes #3732).

Discussion
----------

Update routing.rst. Explain using url() v. path().

The original documentation seemed to imply that, when generating absolute URLs from a template, the argument TRUE should be supplied to the url() method. This change explains that in Twig, one uses the url() method to generate an absolute URL.

Commits
-------

6eac303 Update routing.rst. Explain using url() v. path().
@weaverryan weaverryan closed this Apr 2, 2014
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.

3 participants