coderedcorp / coderedcorp/coderedcms
Use correct site when loading settings
- 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