[rbslack] Polish up the configuration page.

Review Request #6295 — Created Sept. 3, 2014 and submitted

Information

rb-extension-pack
2a2c570...

Reviewers

This adds some additional instructions to the configuration page and
fixes up the labels and widths.

It also makes the channel field optional, since by default,
notifications will go to the channel configured with the "Incoming
WebHook" integration.

Tested with and without a configured channel, and saw it go to the
right place.


reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        rbslack/rbslack/extension.py
        rbslack/rbslack/forms.py
    
    
    
    Tool: Pyflakes
    Processed Files:
        rbslack/rbslack/extension.py
        rbslack/rbslack/forms.py
    
    
  2. 
      
david
  1. Looks good, except your commit message says "rbslash"

  2. 
      
chipx86
chipx86
Review request changed

Status: Closed (submitted)

Loading...