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

Retire l'usage de default_app_config (préparation pour Django 4) #6543

Merged
merged 2 commits into from
Oct 14, 2023

Conversation

Arnaud-D
Copy link
Contributor

Cette PR retire l'usage de default_app_config en préparation pour Django 4 :

Contrôle qualité

  • CI
  • vérifier que le site se lance devrait suffire (les configs des apps sont chargées au démarrage)
  • jouer avec le code pour lancer une notification quelconque et voir que ça fonctionne bien.

* retire l'usage dans zds.notifications.apps (paramètre retiré dans Django 4.1)
* un mécanisme de Django se charge de détecter l'AppConfig par défaut (mécanisme ajouté dans Django 3.2)
@Arnaud-D Arnaud-D added C-Back Concerne le back-end Django hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest labels Oct 14, 2023
@coveralls
Copy link

coveralls commented Oct 14, 2023

Coverage Status

coverage: 88.559% (-0.001%) from 88.56% when pulling a28d37b on Arnaud-D:prep-django4-appconfig into 4d3fefb on zestedesavoir:dev.

Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA OK ✔️

Le site se lance bien et on reçoit bien une notification si quelqu'un commente un de nos billets.

Merci d'avoir mis les liens vers les changelogs de Django pour montrer d'où viennent les changements ! 👍

@philippemilink philippemilink enabled auto-merge (squash) October 14, 2023 16:42
@philippemilink philippemilink merged commit bee600b into zestedesavoir:dev Oct 14, 2023
8 checks passed
@Arnaud-D Arnaud-D deleted the prep-django4-appconfig branch October 15, 2023 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants