New dashboard: remove old legacy dashboard settings (#266)

Author: humitos

dockerfiles/docker-compose.yml

@@ -94,7 +94,6 @@ services:
       - RTD_PUBLIC_DOMAIN
       - RTD_LOGGING_LEVEL
       - RTD_DJANGO_DEBUG
-      - RTD_EXT_THEME_ENABLED
       - RTD_EXT_THEME_DEV_SERVER_ENABLED
       - RTD_FILETREEDIFF_ALL
       # AWS related settings.
@@ -137,7 +136,6 @@ services:
     environment:
       - INIT=${INIT:-}
       - DOCKER_NO_RELOAD
-      - RTD_EXT_THEME_ENABLED
       - RTD_EXT_THEME_DEV_SERVER_ENABLED
       - RTD_PRODUCTION_DOMAIN
       - RTD_PUBLIC_DOMAIN
@@ -196,7 +194,6 @@ services:
       - INIT=${INIT:-}
       - DOCKER_NO_RELOAD
       - NODE_ENV=development
-      - RTD_EXT_THEME_ENABLED
       - RTD_EXT_THEME_DEV_SERVER_ENABLED
     stdin_open: true
     tty: true
@@ -230,7 +227,6 @@ services:
       - COLUMNS=80
       - DOCKER_NO_RELOAD
       - CELERY_LOG_LEVEL=${CELERY_LOG_LEVEL:-INFO}
-      - RTD_EXT_THEME_ENABLED
       - RTD_PRODUCTION_DOMAIN
       - RTD_PUBLIC_DOMAIN
       - RTD_LOGGING_LEVEL
@@ -279,7 +275,6 @@ services:
     environment:
       - DOCKER_NO_RELOAD
       - CELERY_LOG_LEVEL=${CELERY_LOG_LEVEL:-INFO}
-      - RTD_EXT_THEME_ENABLED
       - RTD_PRODUCTION_DOMAIN
       - RTD_PUBLIC_DOMAIN
       - RTD_LOGGING_LEVEL
@@ -341,7 +336,6 @@ services:
       # If COLUMNS is not defined Celery fails at startup
       # https://github.com/celery/celery/issues/5761
       - COLUMNS=80
-      - RTD_EXT_THEME_ENABLED
       - DOCKER_NO_RELOAD
       - CELERY_LOG_LEVEL=${CELERY_LOG_LEVEL:-INFO}
       - RTD_PRODUCTION_DOMAIN

dockerfiles/tasks.py

@@ -63,15 +63,14 @@ def down(c, volumes=False):
     'reload': 'Enable automatic process reloading (default: True)',
     'build': 'Enable automatic building (default: False)',
     'detach': 'Detach mode: run containers in the background (default: False)',
-    'legacy-dashboard': 'Use the old/legacy dashboard (default: False)',
     'scale-build': 'Add additional build instances (default: 1)',
     'http-domain': 'Configure a production domain for HTTP traffic. Subdomains included, '
                    'example.dev implies *.examples.dev for proxito. Example: '
                    '"17b5-139-47-118-243.ngrok.io"',
     'log-level': 'Logging level for the Django application (default: INFO)',
     'django-debug': 'Sets the DEBUG Django setting (default: True)',
 })
-def up(c, search=True, init=False, reload=True, build=False, detach=False, legacy_dashboard=False, scale_build=1, http_domain="", django_debug=True, log_level='INFO'):
+def up(c, search=True, init=False, reload=True, build=False, detach=False, scale_build=1, http_domain="", django_debug=True, log_level='INFO'):
     """Start all the docker containers for a Read the Docs instance"""
     cmd = []
 
@@ -86,9 +85,6 @@ def up(c, search=True, init=False, reload=True, build=False, detach=False, legac
 
     if search:
         cmd.append(f'-f {DOCKER_COMPOSE_SEARCH}')
-    if not legacy_dashboard:
-        cmd.insert(0, 'RTD_EXT_THEME_DEV_SERVER_ENABLED=t')
-        cmd.insert(0, 'RTD_EXT_THEME_ENABLED=t')
     if django_debug:
         cmd.insert(0, 'RTD_DJANGO_DEBUG=t')
     if http_domain: