feat: Add ability to mark a school as a home school

Fixes #157

Author: sman591

app/assets/javascripts/manage/lib/setupDataTables.js

@@ -42,13 +42,14 @@ var setupDataTables = function() {
   });
 
   $('.datatable.schools').DataTable({
-    order: [4, 'desc'],
+    order: [[5, 'desc'], [4, 'desc']],
     columns: [
       { orderable: true, data: 'id', visible: false },
       { orderable: true, data: 'name' },
       { orderable: true, data: 'city' },
       { orderable: true, data: 'state' },
       { orderable: true, data: 'questionnaire_count' },
+      { orderable: true, data: 'home_school' },
     ],
   });
 

app/controllers/manage/dashboard_controller.rb

@@ -49,7 +49,7 @@ def application_activity_data
   def user_distribution_data
     total_stats_data = {}
     total_count = Questionnaire.count
-    rit_count = Questionnaire.where("school_id = \"2304\"").count
+    rit_count = Questionnaire.joins(:school).where("schools.is_home = true").count
     total_stats_data["Incomplete Applications"] = User.without_questionnaire.count
     total_stats_data["Away Applications"] = total_count - rit_count
     total_stats_data["Home Applications"] = rit_count
@@ -117,9 +117,9 @@ def activity_chart_data(types, group_type, range, where_filter = nil)
       when "Applications"
         data = Questionnaire.send("group_by_#{group_type}", :created_at, range: range)
       when "Home Applications"
-        data = Questionnaire.where("school_id = \"2304\"").send("group_by_#{group_type}", :created_at, range: range)
+        data = Questionnaire.joins(:school).where("schools.is_home = true").send("group_by_#{group_type}", :created_at, range: range)
       when "Away Applications"
-        data = Questionnaire.where("school_id != \"2304\"").send("group_by_#{group_type}", :created_at, range: range)
+        data = Questionnaire.joins(:school).where("schools.is_home != true").send("group_by_#{group_type}", :created_at, range: range)
       when "Confirmations"
         data = Questionnaire.where(acc_status: "rsvp_confirmed").send("group_by_#{group_type}", :acc_status_date, range: range)
       when "Denials"

app/controllers/manage/schools_controller.rb

@@ -78,7 +78,7 @@ def perform_merge
 
   def school_params
     params.require(:school).permit(
-      :name, :address, :city, :state
+      :name, :address, :city, :state, :is_home
     )
   end
 

app/datatables/school_datatable.rb

@@ -8,6 +8,7 @@ def view_columns
       city: { source: "School.city" },
       state: { source: "School.state" },
       questionnaire_count: { source: "School.questionnaire_count", searchable: false },
+      home_school: { source: "School.is_home", searchable: false },
     }
   end
 
@@ -21,6 +22,7 @@ def data
         city: record.city,
         state: record.state,
         questionnaire_count: record.questionnaire_count,
+        home_school: record.is_home ? '<span class="badge badge-success">Yes</span>'.html_safe : '<span class="badge badge-secondary">No</span>'.html_safe,
       }
     end
   end

app/views/manage/schools/_form.html.haml

@@ -7,6 +7,7 @@
 
     .form-inputs
       = f.input :name
+      = f.input :is_home
       = f.input :address
       = f.input :city
       = f.input :state

app/views/manage/schools/index.html.haml

@@ -10,6 +10,7 @@
         %th City
         %th State
         %th Count
+        %th Home?
     %tbody
     %tfoot
       %tr
@@ -21,3 +22,4 @@
           %input.form-control.form-control-sm{autocomplete: 'off', placeholder: 'City'}
         %th
           %input.form-control.form-control-sm{autocomplete: 'off', placeholder: 'State'}
+        %th

app/views/manage/schools/show.html.haml

@@ -40,6 +40,12 @@
           = link_to link, target: '_blank' do
             Search in Google Maps
             %span.fa.fa-external-link.icon-space-l-half
+      %dt.col-md-4 Home school
+      %dd.col-md-8
+        - if @school.is_home?
+          %span.badge.badge-success Home school
+        - else
+          %span.badge.badge-secondary Not home school
 
   .col-md-6
     %h4.pb-0 Duplicate Names

config/locales/en.yml

@@ -50,6 +50,8 @@ en:
         type: Bulk emails are sent once, manually. Automated emails are sent upon a desired trigger/event.
         name: A friendly name to recognize this email. Applicants won't see this.
         trigger: Sent automatically when a new or updated applicant matches this criteria. Does not send to anyone already matching this criteria.
+      school:
+        is_home: The "home" school is separated from all other schools on dashboard metrics.
       hackathon_config:
         registration_is_open: Specify that registration is open. This does <strong>not</strong> block registration; it only changes messaging around it.
         event_is_over: Specify that the event is over
@@ -102,6 +104,8 @@ en:
       user:
         email: [email protected]
     labels:
+      school:
+        is_home: Home school
       user:
         is_active: Login access
       hackathon_config:

db/migrate/20190529190615_add_is_home_to_schools.rb

@@ -0,0 +1,5 @@
+class AddIsHomeToSchools < ActiveRecord::Migration[5.2]
+  def change
+    add_column :schools, :is_home, :boolean, default: false
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2019_05_28_155025) do
+ActiveRecord::Schema.define(version: 2019_05_29_190615) do
 
   create_table "active_storage_attachments", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|
     t.string "name", null: false
@@ -241,6 +241,7 @@
     t.datetime "created_at"
     t.datetime "updated_at"
     t.integer "questionnaire_count", default: 0
+    t.boolean "is_home", default: false
   end
 
   create_table "settings", options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t|