coderedcorp / coderedcorp/coderedcms

Use correct site when loading settings

Abierto
#341 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Area: Multisite 🏨 Type: Tech Debt
Lenguaje dominante
Python
Estrellas
765
Forks
154
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

When loading wagtail settings, such as `GoogleApiSettings` or `MailchimpApiSettings` we have a tendency to do these in a place where no `Site` is available and explicitly load the default site. This is a bad practice as it totally breaks the concept of multi-site.

This practice needs to be refactored to get the site from the page or request, and if those are not available, use dependency injection techniques.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Comienza localizando los usos de GoogleApiSettings y MailchimpApiSettings que cargan explícitamente la Site predeterminada; después, comprueba si cada caller tiene contexto de página o de request. Identifica los callers que necesitan dependency injection; se considera terminado cuando los settings resuelven la Site relevante en contextos multisite sin recurrir a la Site predeterminada cuando hay contexto disponible.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
django, python
Área
backend
Tipo de issue
Refactorización
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.