Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: remove "experimental" param from reindex_studio (#37546)

Browse Source
This commit is contained in:
Braden MacDonald
2025-10-29 13:52:06 -07:00
committed by GitHub
parent b033b337a2
commit 4690913a04
2 changed files with 7 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
openedx/core/djangoapps/content/search/api.py
View File

@@ -386,13 +386,13 @@ def init_index(status_cb: Callable[[str], None] | None = None, warn_cb: Callable
if _index_is_empty(STUDIO_INDEX_NAME):
warn_cb(
"The studio search index is empty. Please run ./manage.py cms reindex_studio"
" --experimental [--incremental]"
" [--incremental]"
)
return
if not _is_index_configured(STUDIO_INDEX_NAME):
warn_cb(
"A rebuild of the index is required. Please run ./manage.py cms reindex_studio"
" --experimental [--incremental]"
" [--incremental]"
)
return
status_cb("Index already exists and is configured.")

14
openedx/core/djangoapps/content/search/management/commands/reindex_studio.py
View File

@@ -12,13 +12,15 @@ from ... import api
class Command(BaseCommand):
"""
Build or re-build the search index for courses and libraries (in Studio, i.e. Draft mode)
Build or re-build the Meilisearch search index for courses and libraries in Studio.
This is experimental and not recommended for production use.
This is separate from LMS search features like courseware search or forum search.
"""
# TODO: improve this - see https://github.com/openedx/edx-platform/issues/36868
def add_arguments(self, parser):
parser.add_argument("--experimental", action="store_true")
parser.add_argument("--experimental", action="store_true") # kept for compatibility but ignored.
parser.add_argument("--reset", action="store_true")
parser.add_argument("--init", action="store_true")
parser.add_argument("--incremental", action="store_true")
@@ -31,12 +33,6 @@ class Command(BaseCommand):
if not api.is_meilisearch_enabled():
raise CommandError("Meilisearch is not enabled. Please set MEILISEARCH_ENABLED to True in your settings.")
if not options["experimental"]:
raise CommandError(
"This command is experimental and not recommended for production. "
"Use the --experimental argument to acknowledge and run it."
)
if options["reset"]:
api.reset_index(self.stdout.write)
elif options["init"]: