coderedcorp / coderedcorp/coderedcms

Use correct site when loading settings

Ouverte
#341 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Area: Multisite 🏨 Type: Tech Debt
Langage dominant
Python
Étoiles
765
Forks
154
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Commencez par localiser les utilisations de GoogleApiSettings et MailchimpApiSettings qui chargent explicitement la Site par défaut, puis vérifiez si chaque appelant dispose d’un contexte de page ou de requête. Identifiez les appelants qui nécessitent l’injection de dépendances ; le travail est terminé lorsque les paramètres résolvent la Site pertinente dans les contextes multisite sans revenir à la Site par défaut lorsqu’un contexte est disponible.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
django, python
Domaine
backend
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.