Open options page if Slackbot URL has not been setup

smilledge · Dec 21, 2014 · 34ade62 · 34ade62
1 parent e8298e2
commit 34ade62
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 2 deletions.
diff --git a/css/base.css b/css/base.css
@@ -83,6 +83,10 @@ a, a:hover, a:active, a:visited {
   color: #999;
 }
 
+.required {
+  color: #af1a11;
+}
+
 .button {
   display: block;
   width: 100%;

diff --git a/js/popup.js b/js/popup.js
@@ -24,7 +24,12 @@
         post: function(data) {
             this.isSubmitting = false;
 
-            // TODO: validate channel, message and the web hook url
+            if (!this.options.get('slackbotUrl')) {
+                chrome.tabs.create({
+                    url: chrome.extension.getURL('options.html')
+                });
+                return;
+            }
 
             $.ajax({
                 url: this.options.get('slackbotUrl') + '&channel=' + encodeURIComponent('#' + data.channel),

diff --git a/options.html b/options.html
@@ -15,7 +15,7 @@
             <div id="options">
                 <h1>Slackbot Says Options</h1>
 
-                <h2>Slackbot Url</h2>
+                <h2>Slackbot Url<span class="required">*</span></h2>
 
                 <form action="#" id="options-form" role="form"></form>