coderedcorp / coderedcorp/coderedcms
Use correct site when loading settings
- Lingua principale
- Python
- Stelle
- 765
- Fork
- 154
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
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.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Direzione di ricerca
Inizia individuando gli utilizzi di GoogleApiSettings e MailchimpApiSettings che caricano esplicitamente la Site predefinita, quindi verifica se ogni caller dispone di un contesto di pagina o di request. Identifica i caller che necessitano di dependency injection; il lavoro è completato quando le impostazioni risolvono la Site pertinente nei contesti multisite senza ricorrere alla Site predefinita quando il contesto è disponibile.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- django, python
- Ambito
- backend
- Tipo di issue
- Refactoring
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 35/100