Update dockerfile and readme based on testing with Jeff

Author: maxkfranz

Dockerfile

@@ -2,15 +2,15 @@
 # https://nodejs.org/en/docs/guides/nodejs-docker-webapp/
 # https://github.com/nodejs/docker-node
 
+# Node.js base image
+FROM node:8
+
 # default NODE_ENV for building the app (clientside JS etc.)
 ARG NODE_ENV=production
 
 # default NODE_ENV for running the server
 ENV NODE_ENV=production
 
-# Node.js base image
-FROM node:8
-
 # Create an unprivileged user w/ home directory
 RUN groupadd appuser \
   && useradd --gid appuser --shell /bin/bash --create-home appuser
@@ -23,7 +23,8 @@ WORKDIR /home/appuser/app
 COPY . /home/appuser/app
 
 # Install app dependencies
-RUN npm install
+# Note: must be development so that dev deps are installed
+RUN NODE_ENV=development npm install
 
 # Build project
 RUN npm run clean

README.md

@@ -78,7 +78,7 @@ docker build --build-arg NODE_ENV=production -t app-ui .
 Run the container:
 
 ```
-docker run -it --rm -p 12345:3000 -u "appuser" -e "NODE_ENV=production" --name "app-ui" app-ui:local
+docker run -it --rm -p 12345:3000 -e "NODE_ENV=production" --name "app-ui" app-ui
 ```
 
 Notes: