-
Notifications
You must be signed in to change notification settings - Fork 92
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
Broken in Django 1.10 #58
Comments
See PR #59 comments... |
Hey @tarak, I love your app, any chances of supporting Django 1.10 soon? Thanks! |
Any plans to merge pull request #63 to address this issue? |
What's up here @tarak ? We are looking to migrate to 1.11... |
I suggest to update package and follow https://www.djangoproject.com/download/ Supported Django versions: Django 1.8 LTS, 1.10, 1.11 LTS. |
There's pull request for that. @tarak , if you're no longer interested in maintaining the package perhaps it would be wise to consider passing the maintainership to someone... unless someone forks it |
Just to let you know, I've e-mailed @tarak and asked him to give me commit privs to the repo and to the PyPI package, I'm willing to maintain it. |
@mpasternak @tarak |
I’ve received no reply at all. |
Django 2.0 requires EDIT: here you go #64 |
This project is dead guys, @tarak is just not professional enough to mention us he doesn't want to work on it anymore and too lazy to give the maintainer role to someone else. Seriously, just fork this project and we will all adopt it. |
Hi, I have forked the project and I continue the work in my fork: https://github.com/iplweb/django-password-policies-iplweb Feel free to report any issues, patches, docs improvements also badly needed. I'm using this in a commercial project so I guess I'll maintain my fork for a few years at least. |
Changes in Django 1.10 means MIDDLEWARE_CLASSES now becomes MIDDLEWARE.
password_policies.middleware.PasswordChangeMiddleware now broken:
https://docs.djangoproject.com/en/1.10/topics/http/middleware/#upgrading-pre-django-1-10-style-middleware
All it seems to need is:
Tested, and seems fine.
The text was updated successfully, but these errors were encountered: