Allow sending a test e-mail from the e-mail settings form.

Review Request #5952 — Created June 7, 2014 and submitted

Information

Review Board
release-2.0.x
c9dd052...

Reviewers

When changing the server settings for how to send e-mail, it's nice to give
people the option to test it (so they don't have to create a fake review
request and jump through a bunch of hoops). This change adds a new check-box
that allows users to trigger such an email after the form is saved.

This doesn't cover 100% of the things that can go wrong with sending e-mails
(such as random SPF/DKIM failures or mail servers that don't like the
Sender/From distinction), but it should be good enough to verify that you typed
your SMTP credentials correctly.

This also organizes the e-mail settings into three sections.

Set my e-mail backend to console and saved the form with this check-box set.
Saw that it "sent" an e-mail from the correct Sender (noreply) to my user's
configured e-mail address.

Description From Last Updated

Col: 13 E303 too many blank lines (2)

reviewbotreviewbot

"myself" should probably be "yourself" here. None of our help text reads as the user.

chipx86chipx86
There are no open issues
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        reviewboard/admin/forms.py
      Ignored Files:
    
    
  2. reviewboard/admin/forms.py (Diff revision 1)
     
     
    Show all issues
    Col: 13
     E303 too many blank lines (2)
    
  3. 
      
reviewbot
  1. This is a review from Review Bot.
      Tool: Pyflakes
      Processed Files:
        reviewboard/admin/forms.py
      Ignored Files:
    
    
  2. 
      
david
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        reviewboard/admin/forms.py
      Ignored Files:
    
    
  2. 
      
reviewbot
  1. This is a review from Review Bot.
      Tool: Pyflakes
      Processed Files:
        reviewboard/admin/forms.py
      Ignored Files:
    
    
  2. 
      
chipx86
  1. 
      
  2. reviewboard/admin/forms.py (Diff revision 2)
     
     
    Show all issues

    "myself" should probably be "yourself" here. None of our help text reads as the user.

  3. 
      
david
Review request changed
Status:
Completed
Change Summary:
Pushed to release-2.0.x (748cdb1)
Loading...