Documentation enhancements about django-zxcvbn-password HOT 4 CLOSED

Thanks for your report. Yes a PR would be much appreciated :) Please separate code and documentation in different PRs of course. Just came back from holidays, lot of work to catch up ^^

from django-zxcvbn-password.

Hi -

I actually ended up doing a more manual implementation of zxcvbn rather than using this library. So since I'm not using it, am probably not the best person to do the PR after all. Should be an easy doc tweak though. Thanks!

from django-zxcvbn-password.

So, to get back at this:

• doc about adding zxcvbn_password in INSTALLED_APPS: done thanks to @beruic
• note about upstream docs: I will remove this note since it does not seem to be helpful
• note about form.media: I will update this note to be more clear: including {{ form.media }} in the JS block of your template will in fact include media from all form's fields and widgets. In this app, the only declared media is in the PasswordStrengthInput widget.
• colors not working: indeed the JS code relies on jquery and bootstrap 😕 I will add a note about these requirements in the README.
• guidance about Django's set_password: it's an example among many others, I personally don't use set_password. Usually you just User.objects.create(username=..., password=...) with username and password retrieved from the form cleaned data.

from django-zxcvbn-password.

Thanks @shacker for your feedback!

from django-zxcvbn-password.