Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suppression du paramètre Django SITE_ID #6469

Merged
merged 2 commits into from
Mar 20, 2023

Conversation

philippemilink
Copy link
Member

Ce paramètre est requis lorsqu'on utilise le module django.contrib.sites. Mais on n'utilise plus ce module depuis 2015 (supprimé avec le commit c732cc4).

Il est donc aussi possible de supprimer la table django_site de la base de données.

Contrôle qualité

S'assurer que le site fonctionne (la CI devrait être suffisante).

Ce paramètre est requis lorsqu'on utilise le module
django.contrib.sites. Mais on n'utilise plus ce module depuis 2015
(supprimé avec le commit c732cc4).

Il est donc aussi possible de supprimer la table django_site de la base
de données.
@Situphen Situphen enabled auto-merge (rebase) March 20, 2023 21:32
@Situphen Situphen merged commit 9df6b80 into zestedesavoir:dev Mar 20, 2023
@coveralls
Copy link

Coverage Status

Coverage: 88.31%. Remained the same when pulling 1ae9f43 on philippemilink:remove-siteid into 40f5701 on zestedesavoir:dev.

@philippemilink philippemilink deleted the remove-siteid branch March 22, 2023 17:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants