Errors unless `FORUM_MONGODB_DATABASE` and `FORUM_MONGODB_CLIENT_PARAMETERS` are defined
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Empieza en forum/backends/mongodb/api.py, en get_course_id_by_thread; después, sigue forum/api/threads.py, en get_course_id_by_thread, y la ruta correspondiente de comentarios. Comprueba cómo se gestionan FORUM_MONGODB_DATABASE y FORUM_MONGODB_CLIENT_PARAMETERS cuando faltan, identifica cualquier otro punto de inferencia de course_id y verifica que la configuración ausente devuelva None para que CourseWaffleFlag use el valor predeterminado de todo el sitio.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
By default, we had intended for sites running master to require no changes at all to their configuration to keep the old experience. We currently break this, as @regisb notes in this post. The code is currently doing this because we sometimes need to derive the course_id to know which code path to take, and that's not always available from the request itself.
Is it feasible to add checking to forum.backends.mongodb.api.get_course_id_by_thread_id to explicitly check to see if FORUM_MONGODB_DATABASE and FORUM_MONGODB_CLIENT_PARAMETERS exist, and return None if they don't? That should make the public api call get_course_id_by_thread return None. (And the same for get_course_id_by_comment.) When passing a course_id of None to the CourseWaffleFlag check, it should return the default value for the site as a whole. Which I think will do what we want in a backwards compatible way?
Would that work? Are there other places where we need to try to infer the course_id?
- Lenguaje dominante
- Python
- Estrellas
- 5
- Forks
- 28
- Merge medio
- 12 min
- PR fusionados (30 d)
- 5
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de openedx/forum
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
-
Performance issues with large volumes of discussion dataQuizá libre de nuevo @taimoor-ahmed-1 la tomó hace 391 días y no hay ningún pull request abierto. Abierto
openedx/forum#235 · 10 comentarios · 1 reacción · 1 asignado ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 40/100
-
Remove mocks from edx-platform and use actual functions, instead mock MySQL and ElasticSearch.Abiertocore contributor
Dificultad 4/5 3-5 días Aptitud para principiantes 10/100
Todos los issues de openedx/forum
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
UKGovernmentBEIS/inspect_ai#6512 ·
Los mantenedores suelen responder en 1 día
-
area/c++ area/core kind/bug-report
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
tensorflow/quantum#1140 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
Kav-K/GPTDiscord#501 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
UKGovernmentBEIS/inspect_evals#2863 · 1 comentario ·
Los mantenedores suelen responder en 1 día