Use newer Django auth-related views, and fix a Reset Password e-mail issue.

Review Request #11334 — Created Dec. 15, 2020 and submitted

Information

Review Board
release-4.0.x

Reviewers

Django is getting rid of their legacy auth-related function-based views
and moving to class-based views. Usage of the legacy views emits a
deprecation warning (if enabled). This change updated our URLs to use
the modern views.

This was discovered while investigating an issue with Reset Password
e-mails, where the text would say "Thank you for using !" instead of
"Thank you for using Review Board!" That happened because the e-mail
context is slim, and doesn't have access to Django settings. We now pass
in the product name using one of the class-based view's attributes for
setting e-mail context.

Tested each auth view, making sure nothing regressed.

Reset my password. Verified that the text was correct in the e-mail.

Summary ID
Use newer Django auth-related views, and fix a Reset Password e-mail issue.
Django is getting rid of their legacy auth-related function-based views and moving to class-based views. Usage of the legacy views emits a deprecation warning (if enabled). This change updated our URLs to use the modern views. This was discovered while investigating an issue with Reset Password e-mails, where the text would say "Thank you for using !" instead of "Thank you for using Review Board!" That happened because the e-mail context is slim, and doesn't have access to Django settings. We now pass in the product name using one of the class-based view's attributes for setting e-mail context.
daa05088d107a624bfcfa5195867c45f31479c54
chipx86
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-4.0.x (18b7ef8)
Loading...