incuna / django-user-management

User management model mixins and api views.
BSD 2-Clause "Simplified" License
57 stars 24 forks source link

Fix email verification when `LOGIN_URL` is a url name. #164

Closed maxpeterson closed 7 years ago

maxpeterson commented 7 years ago

@incuna/backend please review / merge. I broke it!

coveralls commented 7 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 87e161bc4775b69ea71cbdd07871c070afd8f5d7 on fix-email-verify-login-url into b409723d94c8d86c1fede5d47754e2c0361047af on master.

adam-thomas commented 7 years ago

Does this still work when LOGIN_URL is a normal URL?

maxpeterson commented 7 years ago

@adam-thomas yes and there is a test for that the original one uses a normal url

adam-thomas commented 7 years ago

:+1: