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

Django-jazzmin - Logout #524

Closed
HamaBarhamou opened this issue Dec 26, 2023 · 7 comments · Fixed by #544
Closed

Django-jazzmin - Logout #524

HamaBarhamou opened this issue Dec 26, 2023 · 7 comments · Fixed by #544
Labels
bug Something isn't working Duplicate A duplicate issue / PR

Comments

@HamaBarhamou
Copy link

hello, here's a conversation that seems to evoke a problem that can be corrected here.

@idan-krayon
Copy link

We implemented this fix on our base code without forking the Jazzmin codebase with the help of the devs in the conversation above. I can confirm there is a problem with the logout button. instead of sending POST /logout, it sends GET /logout.

@Malith-Rukshan
Copy link

Since django-5, you need to do this through a POST request. To fix this, replace logout dropdown at jazzmin base.html with this :-

<form action="{% url 'admin:logout' %}" method="POST">
    {% csrf_token %}
    <button type="submit" class="dropdown-item">
        <i class="fas fa-users mr-2"></i> {% trans 'Log out' %}
    </button>
</form>

Django Admin Logout Fixes
Django Admin Logout Fixed

@jamesgilmorelyst
Copy link

Duplicate issue - #522
PR raised (x2) - #523 / #537

@jamesgilmorelyst jamesgilmorelyst added bug Something isn't working Duplicate A duplicate issue / PR labels Mar 23, 2024
@jamesgilmorelyst
Copy link

jamesgilmorelyst commented Mar 23, 2024

Hey @HamaBarhamou, a PR was merged today (#544) to fix this issue. A new release will be needed to release this fix.

@HamaBarhamou
Copy link
Author

@HamaBarhamou, un PR a été fusionné aujourd'hui ( #544 ) pour résoudre ce problème. Une nouvelle version sera nécessaire pour publier ce correctif.

great, so we can close this problem

@PacificGilly PacificGilly linked a pull request Apr 18, 2024 that will close this issue
@PacificGilly
Copy link
Collaborator

Apologies for the delay, but we've release the fix to resolve the logout issue now:

If you are still supporting an older version of Python (<3.8) or Django (<4) then we also have release a fix for this version as well:

@HamaBarhamou
Copy link
Author

Toutes nos excuses pour le retard, mais nous avons publié le correctif pour résoudre le problème de déconnexion :

Si vous prenez toujours en charge une ancienne version de Python (<3.8) ou de Django (<4), nous avons également publié un correctif pour cette version :

great. glad to have helped

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Duplicate A duplicate issue / PR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants