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

React component of STMO is running in Dev mode #606

Closed
peterbe opened this issue Oct 22, 2018 · 4 comments
Closed

React component of STMO is running in Dev mode #606

peterbe opened this issue Oct 22, 2018 · 4 comments
Assignees
Labels
Milestone

Comments

@peterbe
Copy link

peterbe commented Oct 22, 2018

It's not insecure but definitely slows down significantly.

@peterbe
Copy link
Author

peterbe commented Oct 22, 2018

It's clearly built (bundled and minified) but it's missing the NODE_ENV==production or whatever it's called.

@jezdez jezdez changed the title https://sql.telemetry.mozilla.org is running in Dev mode React component of STMO is running in Dev mode Oct 22, 2018
@rafrombrc rafrombrc added this to the 19 milestone Nov 9, 2018
@rafrombrc rafrombrc added the ready label Nov 9, 2018
@washort
Copy link

washort commented Nov 9, 2018

Not sure where to locate this -- it seems to be built in production mode upstream (as seen on http://redash-preview.netlify.com/ ) and I don't see any changes to webpack or docker config in our fork that would change this. Is this an environment problem in our deploy system?

@peterbe
Copy link
Author

peterbe commented Nov 9, 2018

"deploy system"?? Isn't the minified static asset bundles built before deployment? Perhaps it's word-pickery but in many of projects with React, the yarn run build step is run as part of the Docker build and then the generated minified assets are copied into the container. That means the docker image that is sent to dockerhub has the static assets within them.

@washort
Copy link

washort commented Nov 9, 2018

Ah, I see now. I've added the missing line to our webpack config.

@washort washort closed this as completed in d1a3237 Nov 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants