-
Notifications
You must be signed in to change notification settings - Fork 30
Improve setup doc #58
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
Conversation
aleksblendwerk
commented
Sep 4, 2020
Thanks for these additions, made it more straight-forward to set up the project. Not sure if the docker commands are necessary for the generic use case though.
One thing that could also be improved: in the Running the server locally section, remind the reader that using the built-in Symfony server is only available when APP_ENV is set to dev.
Kind of obvious but I just kept getting There are no commands defined in the "server" namespace even after manually installing symfony/web-server-bundle and that gave me a few 'errr what' moments... (yes, all my fault!)
Not sure if the
dockercommands are necessary for the generic use case though.
You're right, I preferred to put it on because I don't have a front and someone like me who hasn't installed these tools, I took the liberty of leaving it on so I wouldn't have a hard time installing it.
I've added you're reminder.
3e474c5 to
69928d4
Compare
I had a bit of trouble finalizing the configuration of the project because some informations was missing to finish the installation on the frontend side and the configuration of the yaml examples were incorrect, so I preferred to use the command lines to avoid this kind of error.