• 
      

    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:
    Completed
    Change Summary:
    Pushed to release-1.7.x (5b225e5).