Skip to content
This repository was archived by the owner on Nov 27, 2020. It is now read-only.

Fixed the path to config.php in the readme file #605

Closed
wants to merge 2 commits into from
Closed

Fixed the path to config.php in the readme file #605

wants to merge 2 commits into from

Conversation

jaspernbrouwer
Copy link
Contributor

I think the title says it all.

@stof
Copy link
Member

stof commented Nov 12, 2013

I would change it to path/to/project if youo think that path/to/symfony/app is confusing because of the app folder, because it is not the path to symfony (Symfony is in the vendor/symfony/symfony subfolder)

@jaspernbrouwer
Copy link
Contributor Author

Good point! The fix was meant to remove confusion, not to replace it with other confusion ;)

@jaspernbrouwer
Copy link
Contributor Author

I've used dashes (in stead of slashes) to better separate the part the user should replace from the location of the configuration script.

@jaspernbrouwer
Copy link
Contributor Author

Maybe http://localhost/path-to-project-web-folder/config.php is even better, because the web folder is usually configured as document root.
But it may also over-complicate the example.

@ramondelafuente
Copy link

Doesn't this also come back in point 3? That just points to "web/app_dev.php/demo/hello/Fabien"

@jaspernbrouwer
Copy link
Contributor Author

Yes, another reason I think we should stick with http://localhost/path-to-project/web/config.php.

@fabpot fabpot closed this in 977fbee Nov 23, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants