coderedcorp / coderedcorp/coderedcms

Use correct site when loading settings

Offen
#341 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Area: Multisite 🏨 Type: Tech Debt
Vorherrschende Sprache
Python
Sterne
765
Forks
154
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginnen Sie damit, Verwendungen von GoogleApiSettings und MailchimpApiSettings zu lokalisieren, die explizit die standardmäßige Site laden, und verfolgen Sie anschließend, ob jeder Aufrufer über einen Seiten- oder Request-Kontext verfügt. Identifizieren Sie Aufrufer, die Dependency Injection benötigen; abgeschlossen ist die Aufgabe, wenn die Einstellungen in Multisite-Kontexten die relevante Site auflösen, ohne auf die standardmäßige Site zurückzufallen, wenn ein Kontext verfügbar ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
django, python
Bereich
backend
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.