Merge pull request #31 from CS3219-AY2425S1/titus/setup-docker-compose

fix: readme for docker

Author: UnwilledTangent

apps/frontend/README.md

@@ -42,7 +42,7 @@ You can start editing the page by modifying `app/page.tsx`. The page auto-update
 # Navigate to the frontend app directory
 cd apps/frontend
 # Build dockerfile (Ensure that your docker daemon is running beforehand)
-docker build -t frontend -e NEXT_PUBLIC_QUESTION_SERVICE_URL="http://localhost:8080/" -f Dockerfile .
+docker build -t frontend -f Dockerfile .
 ```
 
 Run the backend server locally and visit http://localhost:3000/ to see the frontend application working
@@ -51,7 +51,7 @@ Run the backend server locally and visit http://localhost:3000/ to see the front
 
 ```sh
 # Run the docker image, the -d tag is to run it detached
-docker run -p 3000:3000 -d frontend
+docker run -p 3000:3000 -d frontend -e NEXT_PUBLIC_QUESTION_SERVICE_URL="http://localhost:8080/"
 
 # To see the running container
 docker ps

apps/user-service/README.md

@@ -287,7 +287,7 @@
 # Navigate to the user-service app directory
 cd apps/user-service
 # Build dockerfile after replacing the build arguments (Ensure that your docker daemon is running beforehand)
-docker build -t user-service --build-arg JWT_TOKEN='replace_with_jwt_token' --build-arg DB_CLOUD_URI='replace_with_db_uri_here' -f Dockerfile .
+docker build -t user-service --build-arg JWT_SECRET='replace_with_jwt_secret' --build-arg DB_CLOUD_URI='replace_with_db_uri_here' -f Dockerfile .
 ```
 
 ### Running Docker