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

Remplace la vue de logout par la LogoutView de Django #6272

Merged
merged 2 commits into from
Mar 28, 2022

Conversation

Arnaud-D
Copy link
Contributor

Un des items de #6246.

On utilise une vue fournie par Django plutôt qu'une vue custom. La vue en question permet de stocker la page de redirection comme config, ce que j'ai choisi de faire, afin d'éviter d'avoir un paramètre qui traîne dans le urls.py ou de faire une classe-fille pour juste une ligne.

Contrôle qualité

Se connecter, se déconnecter et constater :

  • qu'on est bien redirigé vers la page d'accueil ;
  • que les cookies de session sont bien nettoyés.

@coveralls
Copy link

coveralls commented Mar 27, 2022

Coverage Status

Coverage decreased (-0.005%) to 87.221% when pulling 6515687 on Arnaud-D:django_logoutview into 59e6187 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.

Rapport de QA

OK ✔️

black est content en local, donc on peut merger.

@philippemilink philippemilink merged commit 2d0324a into zestedesavoir:dev Mar 28, 2022
@Arnaud-D Arnaud-D deleted the django_logoutview branch March 28, 2022 21:41
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