blazer 1.7.7 → 2.6.4

data/app/controllers/blazer/uploads_controller.rb

@@ -0,0 +1,147 @@
+module Blazer
+  class UploadsController < BaseController
+    before_action :ensure_uploads
+    before_action :set_upload, only: [:show, :edit, :update, :destroy]
+
+    def index
+      @uploads = Blazer::Upload.order(:table)
+    end
+
+    def new
+      @upload = Blazer::Upload.new
+    end
+
+    def create
+      @upload = Blazer::Upload.new(upload_params)
+      # use creator_id instead of creator
+      # since we setup association without checking if column exists
+      @upload.creator = blazer_user if @upload.respond_to?(:creator_id=) && blazer_user
+
+      success = params.require(:upload).key?(:file)
+      if success
+        Blazer::Upload.transaction do
+          success = @upload.save
+          if success
+            begin
+              update_file(@upload)
+            rescue CSV::MalformedCSVError, Blazer::UploadError => e
+              @upload.errors.add(:base, e.message)
+              success = false
+              raise ActiveRecord::Rollback
+            end
+          end
+        end
+      else
+        @upload.errors.add(:base, "File can't be blank")
+      end
+
+      if success
+        redirect_to upload_path(@upload)
+      else
+        render_errors @upload
+      end
+    end
+
+    def show
+      redirect_to new_query_path(upload_id: @upload.id)
+    end
+
+    def edit
+    end
+
+    def update
+      original_table = @upload.table
+      @upload.assign_attributes(upload_params)
+
+      success = nil
+      Blazer::Upload.transaction do
+        success = @upload.save
+        if success
+          if params.require(:upload).key?(:file)
+            begin
+              update_file(@upload, drop: original_table)
+            rescue CSV::MalformedCSVError, Blazer::UploadError => e
+              @upload.errors.add(:base, e.message)
+              success = false
+              raise ActiveRecord::Rollback
+            end
+          elsif @upload.table != original_table
+            Blazer.uploads_connection.execute("ALTER TABLE #{Blazer.uploads_table_name(original_table)} RENAME TO #{Blazer.uploads_connection.quote_table_name(@upload.table)}")
+          end
+        end
+      end
+
+      if success
+        redirect_to upload_path(@upload)
+      else
+        render_errors @upload
+      end
+    end
+
+    def destroy
+      Blazer.uploads_connection.execute("DROP TABLE IF EXISTS #{@upload.table_name}")
+      @upload.destroy
+      redirect_to uploads_path
+    end
+
+    private
+
+      def update_file(upload, drop: nil)
+        file = params.require(:upload).fetch(:file)
+        raise Blazer::UploadError, "File is not a CSV" if file.content_type != "text/csv"
+        raise Blazer::UploadError, "File is too large (maximum is 100MB)" if file.size > 100.megabytes
+
+        contents = file.read
+        rows = CSV.parse(contents, converters: %i[numeric date date_time])
+
+        # friendly column names
+        columns = rows.shift.map { |v| v.to_s.encode("UTF-8").gsub("%", " pct ").parameterize.gsub("-", "_") }
+        duplicate_column = columns.find { |c| columns.count(c) > 1 }
+        raise Blazer::UploadError, "Duplicate column name: #{duplicate_column}" if duplicate_column
+
+        column_types =
+          columns.size.times.map do |i|
+            values = rows.map { |r| r[i] }.uniq.compact
+            if values.all? { |v| v.is_a?(Integer) && v >= -9223372036854775808 && v <= 9223372036854775807 }
+              "bigint"
+            elsif values.all? { |v| v.is_a?(Numeric) }
+              "decimal"
+            elsif values.all? { |v| v.is_a?(DateTime) }
+              "timestamptz"
+            elsif values.all? { |v| v.is_a?(Date) }
+              "date"
+            else
+              "text"
+            end
+          end
+
+        begin
+          # maybe SET LOCAL statement_timeout = '30s'
+          # maybe regenerate CSV in Ruby to ensure consistent parsing
+          Blazer.uploads_connection.transaction do
+            Blazer.uploads_connection.execute("DROP TABLE IF EXISTS #{Blazer.uploads_table_name(drop)}") if drop
+            Blazer.uploads_connection.execute("CREATE TABLE #{upload.table_name} (#{columns.map.with_index { |c, i| "#{Blazer.uploads_connection.quote_column_name(c)} #{column_types[i]}" }.join(", ")})")
+            Blazer.uploads_connection.raw_connection.copy_data("COPY #{upload.table_name} FROM STDIN CSV HEADER") do
+              Blazer.uploads_connection.raw_connection.put_copy_data(contents)
+            end
+          end
+        rescue ActiveRecord::StatementInvalid => e
+          raise Blazer::UploadError, "Table already exists" if e.message.include?("PG::DuplicateTable")
+          raise e
+        end
+      end
+
+      def upload_params
+        params.require(:upload).except(:file).permit(:table, :description)
+      end
+
+      def set_upload
+        @upload = Blazer::Upload.find(params[:id])
+      end
+
+      # routes aren't added, but also check here
+      def ensure_uploads
+        render plain: "Uploads not enabled" unless Blazer.uploads?
+      end
+  end
+end

data/app/helpers/blazer/base_helper.rb

@@ -12,9 +12,9 @@ module Blazer
     BLAZER_IMAGE_EXT = %w[png jpg jpeg gif]
 
     def blazer_format_value(key, value)
-      if value.is_a?(Integer) && !key.to_s.end_with?("id") && !key.to_s.start_with?("id")
+      if value.is_a?(Numeric) && !key.to_s.end_with?("id") && !key.to_s.start_with?("id")
         number_with_delimiter(value)
-      elsif value =~ BLAZER_URL_REGEX
+      elsif value.is_a?(String) && value =~ BLAZER_URL_REGEX
         # see if image or link
         if Blazer.images && (key.include?("image") || BLAZER_IMAGE_EXT.include?(value.split(".").last.split("?").first.try(:downcase)))
           link_to value, target: "_blank" do
@@ -29,25 +29,15 @@ module Blazer
     end
 
     def blazer_maps?
-      ENV["MAPBOX_ACCESS_TOKEN"].present?
+      Blazer.mapbox_access_token.present?
     end
 
     def blazer_js_var(name, value)
-      "var #{name} = #{blazer_json_escape(value.to_json)}".html_safe
+      "var #{name} = #{json_escape(value.to_json(root: false))};".html_safe
     end
 
-    JSON_ESCAPE = { '&' => '\u0026', '>' => '\u003e', '<' => '\u003c', "\u2028" => '\u2028', "\u2029" => '\u2029' }
-    JSON_ESCAPE_REGEXP = /[\u2028\u2029&><]/u
-
-    # Prior to version 4.1 of rails double quotes were inadventently removed in json_escape.
-    # This adds the correct json_escape functionality to rails versions < 4.1
-    def blazer_json_escape(s)
-      if Rails::VERSION::STRING < "4.1"
-        result = s.to_s.gsub(JSON_ESCAPE_REGEXP, JSON_ESCAPE)
-        s.html_safe? ? result.html_safe : result
-      else
-        json_escape(s)
-      end
+    def blazer_series_name(k)
+      k.nil? ? "null" : k.to_s
     end
   end
 end

data/app/models/blazer/audit.rb

@@ -1,6 +1,6 @@
 module Blazer
-  class Audit < ActiveRecord::Base
-    belongs_to :user, Blazer::BELONGS_TO_OPTIONAL.merge(class_name: Blazer.user_class.to_s)
-    belongs_to :query, Blazer::BELONGS_TO_OPTIONAL
+  class Audit < Record
+    belongs_to :user, optional: true, class_name: Blazer.user_class.to_s
+    belongs_to :query, optional: true
   end
 end

data/app/models/blazer/check.rb

@@ -1,9 +1,11 @@
 module Blazer
-  class Check < ActiveRecord::Base
-    belongs_to :creator, Blazer::BELONGS_TO_OPTIONAL.merge(class_name: Blazer.user_class.to_s) if Blazer.user_class
+  class Check < Record
+    belongs_to :creator, optional: true, class_name: Blazer.user_class.to_s if Blazer.user_class
     belongs_to :query
 
     validates :query_id, presence: true
+    validate :validate_emails
+    validate :validate_variables, if: -> { query_id_changed? }
 
     before_validation :set_state
     before_validation :fix_emails
@@ -12,6 +14,14 @@ module Blazer
       emails.to_s.downcase.split(",").map(&:strip)
     end
 
+    def split_slack_channels
+      if Blazer.slack?
+        slack_channels.to_s.downcase.split(",").map(&:strip)
+      else
+        []
+      end
+    end
+
     def update_state(result)
       check_type =
         if respond_to?(:check_type)
@@ -57,8 +67,9 @@ module Blazer
       end
 
       # do not notify on creation, except when not passing
-      if (state_was != "new" || state != "passing") && state != state_was && emails.present?
-        Blazer::CheckMailer.state_change(self, state, state_was, result.rows.size, message, result.columns, result.rows.first(10).as_json, result.column_types, check_type).deliver_now
+      if (state_was != "new" || state != "passing") && state != state_was
+        Blazer::CheckMailer.state_change(self, state, state_was, result.rows.size, message, result.columns, result.rows.first(10).as_json, result.column_types, check_type).deliver_now if emails.present?
+        Blazer::SlackNotifier.state_change(self, state, state_was, result.rows.size, message, check_type)
       end
       save! if changed?
     end
@@ -72,7 +83,22 @@ module Blazer
       def fix_emails
         # some people like doing ; instead of ,
         # but we know what they mean, so let's fix it
-        self.emails = emails.gsub(";", ",") if emails.present?
+        # also, some people like to use whitespace
+        if emails.present?
+          self.emails = emails.strip.gsub(/[;\s]/, ",").gsub(/,+/, ", ")
+        end
+      end
+
+      def validate_emails
+        unless split_emails.all? { |e| e =~ /\A\S+@\S+\.\S+\z/ }
+          errors.add(:base, "Invalid emails")
+        end
+      end
+
+      def validate_variables
+        if query.variables.any?
+          errors.add(:base, "Query can't have variables")
+        end
       end
   end
 end

data/app/models/blazer/dashboard.rb

@@ -1,11 +1,15 @@
 module Blazer
-  class Dashboard < ActiveRecord::Base
-    belongs_to :creator, Blazer::BELONGS_TO_OPTIONAL.merge(class_name: Blazer.user_class.to_s) if Blazer.user_class
+  class Dashboard < Record
+    belongs_to :creator, optional: true, class_name: Blazer.user_class.to_s if Blazer.user_class
     has_many :dashboard_queries, dependent: :destroy
     has_many :queries, through: :dashboard_queries
 
     validates :name, presence: true
 
+    def variables
+      queries.flat_map { |q| q.variables }.uniq
+    end
+
     def to_param
       [id, name.gsub("'", "").parameterize].join("-")
     end

data/app/models/blazer/dashboard_query.rb

@@ -1,5 +1,5 @@
 module Blazer
-  class DashboardQuery < ActiveRecord::Base
+  class DashboardQuery < Record
     belongs_to :dashboard
     belongs_to :query
 

data/app/models/blazer/query.rb

@@ -1,6 +1,6 @@
 module Blazer
-  class Query < ActiveRecord::Base
-    belongs_to :creator, Blazer::BELONGS_TO_OPTIONAL.merge(class_name: Blazer.user_class.to_s) if Blazer.user_class
+  class Query < Record
+    belongs_to :creator, optional: true, class_name: Blazer.user_class.to_s if Blazer.user_class
     has_many :checks, dependent: :destroy
     has_many :dashboard_queries, dependent: :destroy
     has_many :dashboards, through: :dashboard_queries
@@ -8,7 +8,8 @@ module Blazer
 
     validates :statement, presence: true
 
-    scope :named, -> { where("blazer_queries.name <> ''") }
+    scope :active, -> { column_names.include?("status") ? where(status: ["active", nil]) : all }
+    scope :named, -> { where.not(name: "") }
 
     def to_param
       [id, name].compact.join("-").gsub("'", "").parameterize
@@ -18,10 +19,35 @@ module Blazer
       name.to_s.sub(/\A[#\*]/, "").gsub(/\[.+\]/, "").strip
     end
 
+    def viewable?(user)
+      if Blazer.query_viewable
+        Blazer.query_viewable.call(self, user)
+      else
+        true
+      end
+    end
+
     def editable?(user)
-      editable = !persisted? || (name.present? && name.first != "*" && name.first != "#") || user == creator
+      editable = !persisted? || (name.present? && name.first != "*" && name.first != "#") || user == try(:creator)
+      editable &&= viewable?(user)
       editable &&= Blazer.query_editable.call(self, user) if Blazer.query_editable
       editable
     end
+
+    def variables
+      # don't require data_source to be loaded
+      variables = Statement.new(statement).variables
+      variables += ["cohort_period"] if cohort_analysis?
+      variables
+    end
+
+    def cohort_analysis?
+      # don't require data_source to be loaded
+      Statement.new(statement).cohort_analysis?
+    end
+
+    def statement_object
+      Statement.new(statement, data_source)
+    end
   end
 end

data/app/models/blazer/record.rb

@@ -0,0 +1,5 @@
+module Blazer
+  class Record < ActiveRecord::Base
+    self.abstract_class = true
+  end
+end

data/app/models/blazer/upload.rb

@@ -0,0 +1,11 @@
+module Blazer
+  class Upload < Record
+    belongs_to :creator, optional: true, class_name: Blazer.user_class.to_s if Blazer.user_class
+
+    validates :table, presence: true, uniqueness: true, format: {with: /\A[a-z0-9_]+\z/, message: "can only contain lowercase letters, numbers, and underscores"}, length: {maximum: 63}
+
+    def table_name
+      Blazer.uploads_table_name(table)
+    end
+  end
+end

data/app/models/blazer/uploads_connection.rb

@@ -0,0 +1,7 @@
+module Blazer
+  class UploadsConnection < ActiveRecord::Base
+    self.abstract_class = true
+
+    establish_connection Blazer.settings["uploads"]["url"] if Blazer.uploads?
+  end
+end

data/app/views/blazer/_nav.html.erb

@@ -5,8 +5,10 @@
     <span class="sr-only">Toggle Dropdown</span>
   </button>
   <ul class="dropdown-menu">
-    <li><%= link_to "Dashboards", dashboards_path %></li>
     <li><%= link_to "Checks", checks_path %></li>
+    <% if Blazer.uploads? %>
+      <li><%= link_to "Uploads", uploads_path %></li>
+    <% end %>
     <li role="separator" class="divider"></li>
     <li><%= link_to "New Query", new_query_path %></li>
     <li><%= link_to "New Dashboard", new_dashboard_path %></li>

data/app/views/blazer/_variables.html.erb

@@ -1,5 +1,15 @@
 <% if @bind_vars.any? %>
-  <form id="bind" method="get" action="<%= action %>" class="form-inline" style="margin-bottom: 10px;">
+  <% var_params = request.query_parameters %>
+  <script>
+    <%= blazer_js_var "timeZone", Blazer.time_zone.tzinfo.name %>
+    var now = moment.tz(timeZone)
+    var format = "YYYY-MM-DD"
+
+    function toDate(time) {
+      return moment.tz(time.format(format), timeZone)
+    }
+  </script>
+  <form id="bind" method="get" action="<%= action %>" class="form-inline" style="margin-bottom: 15px;">
     <% date_vars = ["start_time", "end_time"] %>
     <% if (date_vars - @bind_vars).empty? %>
       <% @bind_vars = @bind_vars - date_vars %>
@@ -10,30 +20,52 @@
     <% @bind_vars.each_with_index do |var, i| %>
       <%= label_tag var, var %>
       <% if (data = @smart_vars[var]) %>
-        <%= select_tag var, options_for_select([[nil, nil]] + data, selected: params[var]), style: "margin-right: 20px; width: 200px; display: none;" %>
+        <%= select_tag var, options_for_select([[nil, nil]] + data, selected: var_params[var]), style: "margin-right: 20px; width: 200px; display: none;" %>
         <script>
           $("#<%= var %>").selectize({
             create: true
           });
         </script>
-      <% else %>
-        <%= text_field_tag var, params[var], style: "width: 120px; margin-right: 20px;", autofocus: i == 0 && !var.end_with?("_at") && !params[var], class: "form-control" %>
-        <% if var.end_with?("_at") %>
-          <script>
-            $("#<%= var %>").daterangepicker({singleDatePicker: true, locale: {format: "YYYY-MM-DD"}, autoUpdateInput: false});
+      <% elsif var.end_with?("_at") || var == "start_time" || var == "end_time" %>
+        <%= hidden_field_tag var, var_params[var] %>
+
+        <div class="selectize-control single" style="width: 200px;">
+          <div id="<%= var %>-select" class="selectize-input" style="display: inline-block;">
+            <span>Select a date</span>
+          </div>
+        </div>
+
+        <script>
+          (function() {
+            var input = $("#<%= var %>")
+            var datePicker = $("#<%= var %>-select")
+            datePicker.daterangepicker({
+              singleDatePicker: true,
+              locale: {format: format},
+              autoUpdateInput: false,
+              autoApply: true,
+              startDate: input.val().length > 0 ? moment.tz(input.val(), timeZone) : now
+            })
             // hack to start with empty date
-            $("#<%= var %>").on("apply.daterangepicker", function(ev, picker) {
-              $(this).val(picker.startDate.format("YYYY-MM-DD"));
-              $(this).change();
-            });
-          </script>
-        <% end %>
+            datePicker.on("apply.daterangepicker", function(ev, picker) {
+              datePicker.find("span").html(toDate(picker.startDate).format("MMMM D, YYYY"))
+              input.val(toDate(picker.startDate).utc().format())
+              submitIfCompleted($("#<%= var %>").closest("form"))
+            })
+            if (input.val().length > 0) {
+              var picker = datePicker.data("daterangepicker")
+              datePicker.find("span").html(toDate(picker.startDate).format("MMMM D, YYYY"))
+            }
+          })()
+        </script>
+      <% else %>
+        <%= text_field_tag var, var_params[var], style: "width: 120px; margin-right: 20px;", autofocus: i == 0 && !var.end_with?("_at") && !var_params[var], class: "form-control" %>
       <% end %>
     <% end %>
 
     <% if date_vars %>
       <% date_vars.each do |var| %>
-        <%= hidden_field_tag var, params[var] %>
+        <%= hidden_field_tag var, var_params[var] %>
       <% end %>
 
       <%= label_tag nil, date_vars.join(" & ") %>
@@ -44,18 +76,10 @@
       </div>
 
       <script>
-        <%= blazer_js_var "timeZone", Blazer.time_zone.tzinfo.name %>
-        var format = "YYYY-MM-DD"
-        var now = moment.tz(timeZone)
-
         function dateStr(daysAgo) {
           return now.clone().subtract(daysAgo || 0, "days").format(format)
         }
 
-        function toDate(time) {
-          return moment.tz(time.format(format), timeZone)
-        }
-
         function setTimeInputs(start, end) {
           $("#start_time").val(toDate(start).utc().format())
           $("#end_time").val(toDate(end).endOf("day").utc().format())
@@ -73,7 +97,8 @@
             },
             startDate: dateStr(29),
             endDate: dateStr(),
-            opens: "right"
+            opens: "right",
+            alwaysShowCalendars: true
           },
           function(start, end) {
             setTimeInputs(start, end)

data/app/views/blazer/check_mailer/failing_checks.html.erb

@@ -1,5 +1,6 @@
 <ul>
   <% @checks.each do |check| %>
+    <%# check queries shouldn't have variables, but in any case, don't pass them to url helpers %>
     <li><%= link_to check.query.name, query_url(check.query_id) %> <%= check.state %></li>
   <% end %>
 </ul>

data/app/views/blazer/check_mailer/state_change.html.erb

@@ -2,6 +2,7 @@
   <head>
   </head>
   <body style="font-family: 'Helvetica Neue', Arial, Helvetica; font-size: 14px; color: #333;">
+    <%# check queries shouldn't have variables, but in any case, don't pass them to url helpers %>
     <p><%= link_to "View", query_url(@check.query_id) %></p>
     <% if @error %>
       <p><%= @error %></p>

data/app/views/blazer/checks/_form.html.erb

@@ -1,19 +1,19 @@
-<% unless @check.respond_to?(:invert) %>
-  <p class="text-muted">Checks are designed to identify bad data. A check fails if there are any results.</p>
-<% end %>
+<%= form_for @check, html: {class: "small-form"} do |f| %>
+  <% unless @check.respond_to?(:check_type) || @check.respond_to?(:invert) %>
+    <p class="text-muted">Checks are designed to identify bad data. A check fails if there are any results.</p>
+  <% end %>
 
-<% if @check.errors.any? %>
-  <div class="alert alert-danger"><%= @check.errors.full_messages.first %></div>
-<% end %>
+  <% if @check.errors.any? %>
+    <div class="alert alert-danger"><%= @check.errors.full_messages.first %></div>
+  <% end %>
 
-<%= form_for @check do |f| %>
   <div class="form-group">
     <%= f.label :query_id, "Query" %>
     <div class="hide">
       <%= f.select :query_id, [], {include_blank: true} %>
     </div>
     <script>
-      <%= blazer_js_var "queries", Blazer::Query.named.order(:name).select("id, name").map { |q| {text: q.name, value: q.id} } %>
+      <%= blazer_js_var "queries", Blazer::Query.active.named.order(:name).select("id, name").map { |q| {text: q.name, value: q.id} } %>
       <%= blazer_js_var "items", [@check.query_id].compact %>
 
       $("#check_query_id").selectize({options: queries, items: items, highlight: false, maxOptions: 100}).parents(".hide").removeClass("hide");
@@ -60,7 +60,15 @@
     <%= f.label :emails %>
     <%= f.text_field :emails, placeholder: "Optional, comma separated", class: "form-control" %>
   </div>
-  <p class="text-muted">Emails are sent when a check starts failing, and when it starts passing again.
+
+  <% if Blazer.slack? %>
+    <div class="form-group">
+      <%= f.label :slack_channels %>
+      <%= f.text_field :slack_channels, placeholder: "Optional, comma separated", class: "form-control" %>
+    </div>
+  <% end %>
+
+  <p class="text-muted">Emails <%= Blazer.slack? ? "and Slack notifications " : nil %>are sent when a check starts failing, and when it starts passing again.
   <p>
     <% if @check.persisted? %>
       <%= link_to "Delete", check_path(@check), method: :delete, "data-confirm" => "Are you sure?", class: "btn btn-danger" %>

data/app/views/blazer/checks/edit.html.erb

@@ -1 +1,3 @@
+<% blazer_title "Edit Check" %>
+
 <%= render partial: "form" %>

data/app/views/blazer/checks/index.html.erb

@@ -1,15 +1,35 @@
 <% blazer_title "Checks" %>
 
-<p style="float: right;"><%= link_to "New Check", new_check_path, class: "btn btn-info" %></p>
-<%= render partial: "blazer/nav" %>
+<div id="header">
+  <div class="pull-right" style="line-height: 34px;">
+    <div class="btn-group">
+      <%= link_to "New Check", new_check_path, class: "btn btn-info" %>
+      <button type="button" class="btn btn-info dropdown-toggle" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false">
+        <span class="caret"></span>
+        <span class="sr-only">Toggle Dropdown</span>
+      </button>
+      <ul class="dropdown-menu">
+        <li><%= link_to "Home", root_path %></li>
+        <% if Blazer.uploads? %>
+          <li><%= link_to "Uploads", uploads_path %></li>
+        <% end %>
+        <li role="separator" class="divider"></li>
+        <li><%= link_to "New Query", new_query_path %></li>
+        <li><%= link_to "New Dashboard", new_dashboard_path %></li>
+      </ul>
+    </div>
+  </div>
 
-<table class="table">
+  <input id="search" type="text" placeholder="Start typing a query or state" style="width: 300px; display: inline-block;" class="search form-control" />
+</div>
+
+<table id="checks" class="table">
   <thead>
     <tr>
       <th>Query</th>
       <th style="width: 10%;">State</th>
       <th style="width: 10%;">Run</th>
-      <th style="width: 20%;">Emails</th>
+      <th style="width: 20%;">Notify</th>
       <th style="width: 15%;"></th>
     </tr>
   </thead>
@@ -19,7 +39,7 @@
         <td><%= link_to check.query.name, check.query %> <span class="text-muted"><%= check.try(:check_type).to_s.gsub("_", " ") %></span></td>
         <td>
           <% if check.state %>
-            <small class="check-state <%= check.state.parameterize("_") %>"><%= check.state.upcase %></small>
+            <small class="check-state <%= check.state.parameterize.gsub("-", "_") %>"><%= check.state.upcase %></small>
           <% end %>
         </td>
         <td><%= check.schedule if check.respond_to?(:schedule) %></td>
@@ -28,6 +48,9 @@
             <% check.split_emails.each do |email| %>
               <li><%= email %></li>
             <% end %>
+            <% check.split_slack_channels.each do |channel| %>
+              <li><%= channel %></li>
+            <% end %>
           </ul>
         </td>
         <td style="text-align: right; padding: 1px;">
@@ -38,3 +61,12 @@
     <% end %>
   </tbody>
 </table>
+
+<script>
+  $("#search").on("keyup", function() {
+    var value = $(this).val().toLowerCase()
+    $("#checks tbody tr").filter( function() {
+      $(this).toggle($(this).text().toLowerCase().indexOf(value) > -1)
+    })
+  }).focus()
+</script>

data/app/views/blazer/checks/new.html.erb

@@ -1 +1,3 @@
+<% blazer_title "New Check" %>
+
 <%= render partial: "form" %>