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'argument obsolète "providing_args" pour la création des signaux (préparation pour Django 4) #6533

Merged
merged 1 commit into from
Oct 1, 2023

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Oct 1, 2023

Cette PR concoure à la préparation du passage à Django 4.X.

Certains de nos signaux sont crées en fournissant l'argument providing_args. Cet argument est déjà obsolète dans la version 3.2 de Django et est supprimé dans la 4.0.

J'ai effectué la correction suggérée, à savoir mettre l'information associée à cet argument dans un commentaire pour documenter.

Contrôle qualité

  • CI
  • lancer le site et vérifier rapidement que ça marche normalement.

@Arnaud-D Arnaud-D added the C-Back Concerne le back-end Django label Oct 1, 2023
@coveralls
Copy link

Coverage Status

coverage: 88.413%. remained the same when pulling 100b616 on Arnaud-D:prepa-django4-signals into e127949 on zestedesavoir:dev.

Copy link
Member

@Situphen Situphen 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 ! J'ai testé que les pings fonctionnaient bien

@Situphen Situphen merged commit fb54169 into zestedesavoir:dev Oct 1, 2023
8 checks passed
@Arnaud-D Arnaud-D deleted the prepa-django4-signals branch October 1, 2023 13:32
@Arnaud-D Arnaud-D added the hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest label Oct 8, 2023
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