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

ProfilerStorage configuration #6024

Closed
wants to merge 1 commit into from
Closed

Conversation

xavierleune
Copy link
Contributor

The ProfilerStorage other than FileProfilerStorage have been removed in this PR : symfony/symfony#16018

The ProfilerStorage other than FileProfilerStorage have been removed in this PR : symfony/symfony#16018
@xabbuh
Copy link
Member

xabbuh commented Dec 17, 2015

Actually, you can still change the storage path using the dsn option. I think this should still be explained (but it's however a good idea to also hint to the storage service configuration option).

@javiereguiluz
Copy link
Member

@xavierleune thanks for your contribution. I'm trying to finish this pull request in #6090. I've reused your original commit, so you'll get full credit of your work when this is merged. Thanks!

@xabbuh xabbuh closed this Dec 30, 2015
@xavierleune
Copy link
Contributor Author

@javiereguiluz sorry for the delate and thanks for your work !

xabbuh added a commit that referenced this pull request Feb 7, 2016
… javiereguiluz)

This PR was merged into the 3.0 branch.

Discussion
----------

Reworded the article about profiler storage

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | 3.0+
| Fixed tickets | -

This finishes #6024.

Commits
-------

bcbb87f Fixed the Symfony version number
bcc5634 Finished the rewording of the "profiler storage" article
32a63d8 ProfilerStorage configuration
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