-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Improved routing section #11245
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
Improved routing section #11245
Conversation
routing.rst
Outdated
|
||
$ composer require annotations | ||
|
||
.. note:: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wouldn't add such a detailed note ... maybe we can move this to a phrase wrapped in parenthesis after the "... run this command once in your app to add support for them:"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be fixed in 3.4 too
security/form_login_setup.rst
Outdated
@@ -74,6 +74,51 @@ class that processes the login submit and 4) updates the main security config fi | |||
} | |||
} | |||
|
|||
Edit the ``security.yml`` file in order to allow access for anyone to the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
security.yaml
security/form_login_setup.rst
Outdated
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" | ||
xmlns:srv="http://symfony.com/schema/dic/services" | ||
xsi:schemaLocation="http://symfony.com/schema/dic/services | ||
http://symfony.com/schema/dic/services/services-1.0.xsd"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
HttpS for xsd url
@OskarStark If this needs to be fixed in 3.4 I'll do it in another PR. |
Jules, I'm sorry it took us so long to merge this. Thanks for your contribution! |
Continuation of #11113 in 4.2.