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 2.1 & 2.2 compatibility? #125

Closed
jhancia opened this issue May 19, 2019 · 4 comments · Fixed by #154
Closed

Django 2.1 & 2.2 compatibility? #125

jhancia opened this issue May 19, 2019 · 4 comments · Fixed by #154

Comments

@jhancia
Copy link

jhancia commented May 19, 2019

Is this package compatible with Django 2.1 & 2.2? Anyone using it in production? If so, we should add it to tox.ini and travis.yml

@sillycube
Copy link

sillycube commented Sep 13, 2019

Yes, I am using python 3.7, Django 2.2, jQuery 2.1.4. It works. But the documentation needs to be updated:

  1. settings.TEMPLATE_CONTEXT_PROCESSORS - 'django.core.context_processors.request' is replaced by 'django.template.context_processors.request' according to the documentation here.

  2. url(r'session_security/', include('session_security.urls')) is changed in django 2.2. Use path('session_security/', include('session_security.urls')),

@mpasternak
Copy link
Contributor

Ping me in a pull request and I’ll review and merge. Thanks.

@its4net
Copy link

its4net commented Sep 10, 2021

Not working for me in Django 2.2, any tips? I followed changes above. User get logged out after time regardless of activity and the popup warning never shows.

@JanMalte
Copy link
Contributor

JanMalte commented Feb 8, 2022

@its4net do you get any JavaScript errors in your browser debug console?

Did you included the session_security/all.html template?

And did you loaded jQuery in any way?

{# Load SessionSecurity javascript 'class', jquery should be loaded - by you - at this point #}
<script type="text/javascript" src="{% static 'session_security/script.js' %}"></script>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants