Titleize windows type short_names across codebase

ADULT → Adult, CHILDREN → Children, COMBINED → Combined.
Updates seeds, factories, controller lookups, rake tasks, and specs
to match production data.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: maebeale

app/controllers/monthly_reports_controller.rb

@@ -209,7 +209,7 @@ def find_workshop_logs
   end
 
   def find_combined_workshop_logs(agency_id)
-    combined_windows_type = WindowsType.where(short_name: "COMBINED").first
+    combined_windows_type = WindowsType.where(short_name: "Combined").first
     @combined_workshop_logs = current_user.organization_workshop_logs(
     @report.date, combined_windows_type, agency_id)
   end

app/controllers/workshop_log_creation_wizard_controller.rb

@@ -8,7 +8,7 @@ def show
     authorize! :workshop_log_creation_wizard, to: :show?
     @user = current_user
     @agencies = current_user.organizations
-    windows_type_id = params[:windows_type_id] || WindowsType.where(short_name: "COMBINED")
+    windows_type_id = params[:windows_type_id] || WindowsType.where(short_name: "Combined")
     @windows_type = WindowsType.find(windows_type_id) if windows_type_id
     send(step)
     render_wizard

app/controllers/workshop_logs_controller.rb

@@ -192,7 +192,7 @@ def set_form_variables
     # @files = MediaFile.where(["workshop_log_id = ?", @workshop_log.id])
 
     @windows_type_id = params[:windows_type_id].presence || @workshop.windows_type_id ||
-      WindowsType.where(short_name: "COMBINED").last.id
+      WindowsType.where(short_name: "Combined").last.id
     form = FormBuilder.where(windows_type_id: @windows_type_id)
                       .first&.forms.first # because there's only one form per form_builder
     if form

app/controllers/workshops_controller.rb

@@ -45,7 +45,7 @@ def summary
     @total_ongoing    = logs.reduce(0) { |sum, l| sum += l.num_ongoing }
     @total_first_time = logs.reduce(0) { |sum, l| sum += l.num_first_time }
 
-    combined_windows_type = WindowsType.where(short_name: "COMBINED").first
+    combined_windows_type = WindowsType.where(short_name: "Combined").first
     @combined_workshop_logs = current_user.organization_workshop_logs(
       @report.date, combined_windows_type, current_user.agency_id
     )

db/seeds.rb

@@ -66,11 +66,11 @@ def find_or_create_by_name!(klass, name, **attrs, &block)
 
 puts "Creating WindowsTypes…"
 adult_type = WindowsType.where(name: "ADULT WINDOWS")
-                        .first_or_create!(legacy_id: 1, short_name: "ADULT")
+                        .first_or_create!(legacy_id: 1, short_name: "Adult")
 childrens_type = WindowsType.where(name: "CHILDREN'S WINDOWS")
-                            .first_or_create!(legacy_id: 2, short_name: "CHILDREN")
+                            .first_or_create!(legacy_id: 2, short_name: "Children")
 combined_type = WindowsType.where(name: "ADULT & CHILDREN COMBINED (FAMILY) WINDOWS")
-                           .first_or_create!(legacy_id: 3, short_name: "COMBINED")
+                           .first_or_create!(legacy_id: 3, short_name: "Combined")
 
 puts "Creating FormBuilders…"
 FormBuilder.where(name: "Adult Monthly Report", windows_type: adult_type).first_or_create!(id: 4)

db/seeds/dummy_dev_seeds.rb

@@ -4,9 +4,9 @@
 pending_status = OrganizationStatus.find_by!(name: "Pending")
 suspended_status = OrganizationStatus.find_by!(name: "Suspended")
 
-adult_wt = WindowsType.find_by!(short_name: "ADULT")
-children_wt = WindowsType.find_by!(short_name: "CHILDREN")
-combined_wt = WindowsType.find_by!(short_name: "COMBINED")
+adult_wt = WindowsType.find_by!(short_name: "Adult")
+children_wt = WindowsType.find_by!(short_name: "Children")
+combined_wt = WindowsType.find_by!(short_name: "Combined")
 
 [
   { name: "1736 Family Crisis Center", organization_status: inactive_status, windows_type: adult_wt },

lib/tasks/story_data.rake

@@ -19,7 +19,7 @@ namespace :story_data do
         spotlighted_facilitator_id: nil,
         story_idea_id:         nil,
         website_url:           resource.url,
-        windows_type_id:       resource.windows_type_id || WindowsType.where(short_name: "COMBINED").first.id,
+        windows_type_id:       resource.windows_type_id || WindowsType.where(short_name: "Combined").first.id,
         workshop_id:           resource.workshop_id,
         youtube_url:           nil, # resources don't store this, add mapping if needed
         created_at:            resource.created_at,

spec/factories/windows_types.rb

@@ -5,17 +5,17 @@
 
     trait :adult do
       name { "ADULT WINDOWS" }
-      short_name { "ADULT" }
+      short_name { "Adult" }
     end
 
     trait :children do
       name { "CHILDREN'S WINDOWS" }
-      short_name { "CHILDREN" }
+      short_name { "Children" }
     end
 
     trait :combined do
       name { "ADULT & CHILDREN COMBINED (FAMILY) WINDOWS" }
-      short_name { "COMBINED" }
+      short_name { "Combined" }
     end
   end
 end

spec/models/workshop_spec.rb

@@ -63,7 +63,7 @@
     it 'returns title + windows type (when present) + # + id' do
       record = create(:workshop, title: 'The best workshop in the world', windows_type: create(:windows_type, :adult))
 
-      expect(record.type_name).to eq "The best workshop in the world (ADULT) ##{record.id}"
+      expect(record.type_name).to eq "The best workshop in the world (Adult) ##{record.id}"
     end
 
     it 'omits the windows type part when there is no windows_type' do

spec/requests/stories_spec.rb

@@ -58,8 +58,8 @@
 
         let(:org_alpha) { create(:organization, name: "Alpha Org") }
         let(:org_zulu) { create(:organization, name: "Zulu Org") }
-        let(:wt_adult) { create(:windows_type, short_name: "ADULT") }
-        let(:wt_children) { create(:windows_type, short_name: "CHILDREN") }
+        let(:wt_adult) { create(:windows_type, short_name: "Adult") }
+        let(:wt_children) { create(:windows_type, short_name: "Children") }
         let(:ws_art) { create(:workshop, title: "Art Workshop") }
         let(:ws_music) { create(:workshop, title: "Music Workshop") }