• 
      

    [rbslack] Don't cause a 500 if notification fails.

    Review Request #6354 — Created Sept. 21, 2014 and submitted

    Information

    rb-extension-pack
    c847334...

    Reviewers

    In the case where something fails when trying to send to the slack.com webhook,
    we shouldn't show the user an error 500. This change catches all exceptions and
    logs them.
    
     
    reviewbot
    1. Tool: PEP8 Style Checker
      Processed Files:
          rbslack/rbslack/extension.py
      
      
      
      Tool: Pyflakes
      Processed Files:
          rbslack/rbslack/extension.py
      
      
    2. 
        
    chipx86
    1. Ship It!

    2. 
        
    david
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to master (36f0345)