From 0874b2f5c7928cff668c090b0f783f1540ff1ca8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9gis=20Behmo?= <regis@behmo.com>
Date: Tue, 29 Oct 2024 17:39:06 +0100
Subject: [PATCH] code review: first round of comments

---
 tutor/templates/apps/openedx/config/cms.env.yml              | 1 -
 tutor/templates/apps/openedx/config/lms.env.yml              | 3 ---
 tutor/templates/apps/openedx/settings/partials/common_lms.py | 1 +
 tutor/templates/dev/docker-compose.yml                       | 2 ++
 tutor/templates/k8s/volumes.yml                              | 2 +-
 5 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/tutor/templates/apps/openedx/config/cms.env.yml b/tutor/templates/apps/openedx/config/cms.env.yml
index 53e99b0c2d..8eb7b462d8 100644
--- a/tutor/templates/apps/openedx/config/cms.env.yml
+++ b/tutor/templates/apps/openedx/config/cms.env.yml
@@ -9,7 +9,6 @@ FEATURES:
   {{ patch("cms-env-features")|indent(2) }}
   CERTIFICATES_HTML_VIEW: true
   PREVIEW_LMS_BASE: "{{ PREVIEW_LMS_HOST }}"
-  ENABLE_COURSEWARE_INDEX: true
   ENABLE_CSMH_EXTENDED: false
   ENABLE_LEARNER_RECORDS: false
   ENABLE_LIBRARY_INDEX: true
diff --git a/tutor/templates/apps/openedx/config/lms.env.yml b/tutor/templates/apps/openedx/config/lms.env.yml
index 565820c81b..bc9927f23c 100644
--- a/tutor/templates/apps/openedx/config/lms.env.yml
+++ b/tutor/templates/apps/openedx/config/lms.env.yml
@@ -9,10 +9,7 @@ FEATURES:
   {{ patch("lms-env-features")|indent(2) }}
   CERTIFICATES_HTML_VIEW: true
   PREVIEW_LMS_BASE: "{{ PREVIEW_LMS_HOST }}"
-  ENABLE_COURSE_DISCOVERY: true
-  ENABLE_COURSEWARE_SEARCH: true
   ENABLE_CSMH_EXTENDED: false
-  ENABLE_DASHBOARD_SEARCH:  true
   ENABLE_COMBINED_LOGIN_REGISTRATION: true
   ENABLE_GRADE_DOWNLOADS: true
   ENABLE_LEARNER_RECORDS: false
diff --git a/tutor/templates/apps/openedx/settings/partials/common_lms.py b/tutor/templates/apps/openedx/settings/partials/common_lms.py
index 727871be3d..befd6d6d2f 100644
--- a/tutor/templates/apps/openedx/settings/partials/common_lms.py
+++ b/tutor/templates/apps/openedx/settings/partials/common_lms.py
@@ -40,6 +40,7 @@
 # Enable search features
 FEATURES["ENABLE_COURSE_DISCOVERY"] = True
 FEATURES["ENABLE_COURSEWARE_SEARCH"] = True
+FEATURES["ENABLE_DASHBOARD_SEARCH"] = True
 
 # Create folders if necessary
 for folder in [DATA_DIR, LOG_DIR, MEDIA_ROOT, STATIC_ROOT, ORA2_FILEUPLOAD_ROOT]:
diff --git a/tutor/templates/dev/docker-compose.yml b/tutor/templates/dev/docker-compose.yml
index e65bde5a5f..5ef5387d39 100644
--- a/tutor/templates/dev/docker-compose.yml
+++ b/tutor/templates/dev/docker-compose.yml
@@ -32,6 +32,7 @@ services:
     ports:
         - "8001:8000"
 
+  {% if RUN_MEILISEARCH -%}
   meilisearch:
     ports:
       - "127.0.0.1:7700:7700"
@@ -39,6 +40,7 @@ services:
       default:
         aliases:
           - "{{ MEILISEARCH_PUBLIC_URL.split('://')[1] }}"
+  {%- endif %}
 
   # Additional service for watching theme changes
   watchthemes:
diff --git a/tutor/templates/k8s/volumes.yml b/tutor/templates/k8s/volumes.yml
index 1535a1d900..aa31555ba0 100644
--- a/tutor/templates/k8s/volumes.yml
+++ b/tutor/templates/k8s/volumes.yml
@@ -28,7 +28,7 @@ spec:
     - ReadWriteOnce
   resources:
     requests:
-      storage: 2Gi
+      storage: 5Gi
 {% endif %}
 {% if RUN_MONGODB %}
 ---