e-mission / nrel-openpath-deploy-configs

Configurations for current OpenPATH deployments, published for transparency
BSD 3-Clause "New" or "Revised" License
2 stars 9 forks source link

🐛 🚑 UUE-Openpath Patch #34

Closed Abby-Wheelis closed 10 months ago

Abby-Wheelis commented 10 months ago

There is an issue with the config preventing it from being deployed, hopefully we can get that fixed here.

In scrutinizing the config file, I found that I had forgotten to copy over the new deployment_partner_name into the translated_text section -- but that should not be breaking the config.

Abby-Wheelis commented 10 months ago
Screenshot 2023-09-06 at 10 44 45 AM

Join page is working on testing! Checking the config's function in this way was how I discovered that I hadn't updated the translated text partner name

shankari commented 10 months ago

I tested this by applying this patch to the docker-compose file in op-admin-dashboard

diff --git a/docker-compose-dev.yml b/docker-compose-dev.yml
index 75f8df2..02d7354 100644
--- a/docker-compose-dev.yml
+++ b/docker-compose-dev.yml
@@ -12,11 +12,11 @@ services:
       DASH_DEBUG_MODE: "True"
       DASH_SILENCE_ROUTES_LOGGING: "False"
       DASH_SERVER_PORT: 8050
       DB_HOST: db
       WEB_SERVER_HOST: 0.0.0.0
       SERVER_BRANCH: master
-      CONFIG_PATH: "https://raw.githubusercontent.com/e-mission/nrel-openpath-deploy-configs/main/configs/"
-      STUDY_CONFIG: "stage-program"
+      CONFIG_PATH: "https://github.com/Abby-Wheelis/nrel-openpath-deploy-configs/tree/Abby-Wheelis-uue-patch/configs/"
+      STUDY_CONFIG: "uue"
       AUTH_TYPE: "basic" # the other option is cognito
       REACT_VERSION: "18.2.0"
     networks:

When I had left out the / at the end of the CONFIG_PATH, this failed with the same error as reported by IT. When I added it back, the dashboard launched and I could log in.