8000 Symfony 4 Security: Added note, how to install profiler bundle by aurelijusb · Pull Request #9537 · symfony/symfony-docs · GitHub
[go: up one dir, main page]

Skip to content

Symfony 4 Security: Added note, how to install profiler bundle #9537

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 1 commit into from
May 5, 2018

Conversation

aurelijusb
Copy link
Contributor
@aurelijusb aurelijusb commented Apr 1, 2018

There is example picture (_images/security/anonymous_wdt.png) with profiler bundle,
but when creating Symfony 4 project with flexprofiler bundle is not included by default.

Therefore for new Symfony 4 users it is hard to follow documentation.

There is example picture (`_images/security/anonymous_wdt.png`) with `profiler` bundle, but when creating Symfony 4 project with `flex` – `profiler` bundle is not included by default.
Therefore for new Symfony 4 users it is hard to follow documentation.
@aurelijusb aurelijusb changed the title Added example, how to install profiler bundle Symfony 4 Security: Added note, how to install profiler bundle Apr 1, 2018
@wouterj wouterj merged commit 45c5399 into symfony:4.0 May 5, 2018
wouterj added a commit that referenced this pull request May 5, 2018
…undle (aurelijusb)

This PR was merged into the 4.0 branch.

Discussion
----------

Symfony 4 Security:  Added note, how to install profiler bundle

There is example picture (`_images/security/anonymous_wdt.png`) with `profiler` bundle,
but when creating Symfony 4 project with `flex` – `profiler` bundle is not included by default.

Therefore for new Symfony 4 users it is hard to follow documentation.

Commits
-------

45c5399 Added example, how to install profiler bundle
@wouterj
Copy link
Member
wouterj commented May 5, 2018

Hi @aurelijusb!

Thanks for submitting this PR. I think this definitely can be confusing for Flex users, so I've merged it directly. In 06bb097, I've moved it a bit down (otherwise, the single sentence below it wouldn't be noticed anymore) and slightly reworded it.

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
0