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

Added info how to disable the mock session #8387

Closed
wants to merge 2 commits into from

Conversation

ThomasLandauer
Copy link
Contributor

Should be explained somewhere...

Should be explained somewhere...
@xabbuh
Copy link
Member

xabbuh commented Sep 13, 2017

I can see that this information is somehow useful. But I don't think that this file is the right place as it describes the usage of the HttpFoundation component without using the full framework. Thus, you probably won't have such a config_test.yml file.

@ThomasLandauer
Copy link
Contributor Author

Well, since you didn't suggest a better place, I guess there is none :-)
What about just adding: "If you're using the full framework..."

Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

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

I can't find any better place to put this, so let's merge it as is. Thanks!

@xabbuh
Copy link
Member

xabbuh commented Jan 9, 2018

#8994 proposes to explain the mock file handler too

@javiereguiluz
Copy link
Member

@xabbuh I'm confused by all this. What do you propose to do to solve this and #8994?

@xabbuh
Copy link
Member

xabbuh commented Jan 9, 2018

I am not sure yet. But I am also a bit confused about the changes themselves. The text talks about disabling the mock session storage, but if I don't miss anything, the following config example does the exact opposite.

@javiereguiluz
Copy link
Member

Let's close this because in the configuration reference of the Framework Bundle we already explain this option (https://symfony.com/doc/current/reference/configuration/framework.html#storage-id) so it's better to not mention in the component docs too. Thanks!

@ThomasLandauer ThomasLandauer deleted the patch-11 branch September 27, 2021 17:06
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.

4 participants