Fix review request ID in email subjects.

Review Request #4032 — Created April 7, 2013 and submitted

Information

Review Board
release-1.7.x

Reviewers

Fix review request ID in email subjects.

We were using the id in email subjects instead of display_id. Very simple fix.

 
Description From Last Updated

Col: 80 E501 line too long (92 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (108 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (108 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (83 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (81 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (83 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (81 > 79 characters)

reviewbotreviewbot
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        reviewboard/notifications/email.py
      Ignored Files:
    
    
    1. These were already broken.
  2. reviewboard/notifications/email.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (92 > 79 characters)
    
    1. Can we fix these? These are the only comments I had on it.
  3. reviewboard/notifications/email.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (108 > 79 characters)
    
  4. reviewboard/notifications/email.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (108 > 79 characters)
    
  5. 
      
david
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        reviewboard/notifications/email.py
      Ignored Files:
    
    
  2. reviewboard/notifications/email.py (Diff revision 2)
     
     
    Show all issues
    Col: 80
     E501 line too long (83 > 79 characters)
    
  3. reviewboard/notifications/email.py (Diff revision 2)
     
     
    Show all issues
    Col: 80
     E501 line too long (81 > 79 characters)
    
  4. reviewboard/notifications/email.py (Diff revision 2)
     
     
    Show all issues
    Col: 80
     E501 line too long (83 > 79 characters)
    
  5. reviewboard/notifications/email.py (Diff revision 2)
     
     
    Show all issues
    Col: 80
     E501 line too long (81 > 79 characters)
    
  6. 
      
david
reviewbot
  1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
        reviewboard/notifications/email.py
      Ignored Files:
    
    
  2. 
      
chipx86
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-1.7.x (5b225e5).
Loading...