-
Notifications
You must be signed in to change notification settings - Fork 248
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
Production Mode vs Developer Mode #88
Comments
It is very easy to change. Use Then you will get errors. Simply run |
@JackWormUK Yes I am aware how to change the deploy mode in Magento, and that is what I did to get it working. The purpose of this issue was to request the ability to set this from the beginning as to avoid needing to perform this extra step. In install-magento line 36 it explicitly sets production mode. I just don't see why this is necessary as this is advertised as a local development environment. If anything it should explicitly set But the better option I believe would be to provide an optional prompt, ENV variable, or some other way to configure this before running the install script that will tell it which mode to set. |
@rain2o I don't disagree with you. It would be nice if at the very end it switched over to developer mode using the 2 command above. I only commented here to help out others who may wish to switch over to developer mode by giving them the quick and easy way to do it. |
@JackWormUK I has break all layout after try this command to set to develop mode |
But i have so say, this project is the best docker for osx i try!!! |
As this project is advertised as "Docker Image to Magento 2 local development in Mac, Linux and Windows" I am wondering why it is setup to run in
production
mode? After running the initial installation I had to go in and clear all compilation and static content and switch todeveloper
mode so I could begin development. Is there a particular reason this is setup for production mode?Having the option to set the mode could be a solution if it depends on the user's use-case.
The text was updated successfully, but these errors were encountered: