coderedcorp / coderedcorp/coderedcms

Use correct site when loading settings

Aperta
#341 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Area: Multisite 🏨 Type: Tech Debt
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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.