Merge pull request #31 from yurenju/feat/docker-undefined

fix: undefined value in init-db script
moda-gov-tw · Sep 28, 2023 · 23a5806 · 23a5806
2 parents 4fd398f + e25a535
commit 23a5806
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 7 deletions.
diff --git a/apps/docker/project.json b/apps/docker/project.json
@@ -6,10 +6,6 @@
   "targets": {
     "build": {
       "dependsOn": [
-        {
-          "target": "copy-env-file",
-          "projects": ["server"]
-        },
         {
           "target": "build",
           "projects": ["web", "server"]

diff --git a/apps/server/project.json b/apps/server/project.json
@@ -21,7 +21,8 @@
       "configurations": {
         "development": {},
         "production": {}
-      }
+      },
+      "dependsOn": ["setup-db"]
     },
     "serve": {
       "executor": "@nx/js:node",
@@ -74,13 +75,13 @@
       }
     },
     "setup-db": {
-      "command": "node apps/server/scripts/setup-db.js"
+      "command": "node apps/server/scripts/setup-db.js",
+      "dependsOn": ["copy-env-file"]
     },
     "start-db": {
       "command": "node apps/server/scripts/start-db.js"
     },
     "copy-env-file": {
-      "dependsOn": ["setup-db"],
       "command": "node apps/server/scripts/copy-env-file.js"
     }
   },