• 
      

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

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

    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.

    Commits

    Files