Opened 7 years ago

Closed 7 years ago

#7274 closed (fixed)

Reset password view shouldn't require sites application

Reported by: joshrl <jrookeley@…> Owned by: nobody
Component: Contrib apps Version: master
Severity: Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

The reset password view (django.contrib.auth.views.password_reset) requires that the django.contrib.sites application is installed. Perhaps there is a good reason for this, but I couldn't think of one so I created a patch to use RequestSite, if sites isn't installed.

Attachments (1)

reset_password_site_not_required_r7540.diff (914 bytes) - added by joshrl <jrookeley@…> 7 years ago.

Download all attachments as: .zip

Change History (3)

Changed 7 years ago by joshrl <jrookeley@…>

comment:1 Changed 7 years ago by programmerq

  • Triage Stage changed from Unreviewed to Ready for checkin

comment:2 Changed 7 years ago by adrian

  • Resolution set to fixed
  • Status changed from new to closed

(In [7748]) Fixed #7274 -- Improved the django.contrib.auth password_reset() view not to require the Sites framework. Thanks, joshrl

Note: See TracTickets for help on using tickets.
Back to Top