From 4e8adaa0ea59fdcaa7753bf1a6b2eea5f177853f Mon Sep 17 00:00:00 2001 From: Fabio Lima Date: Sun, 15 Jun 2025 09:14:35 -0700 Subject: [PATCH 1/2] Add local port mapping for Postgres and enhance logging in data rake task --- docker-compose.yml | 1 + lib/tasks/data.rake | 3 +++ 2 files changed, 4 insertions(+) diff --git a/docker-compose.yml b/docker-compose.yml index 26e712c3..d1c48363 100644 --- a/docker-compose.yml +++ b/docker-compose.yml @@ -44,6 +44,7 @@ services: - "35432" ports: - "35432" + - "127.0.0.1:35432:35432" # Uncomment to access this containers Postgres instance via port 5432 # - "127.0.0.1:5432:5432" environment: diff --git a/lib/tasks/data.rake b/lib/tasks/data.rake index 145edaeb..d899223c 100644 --- a/lib/tasks/data.rake +++ b/lib/tasks/data.rake @@ -28,6 +28,7 @@ namespace :data do "#{header} #{msg}\n" end + attention_logger = ActiveSupport::Logger.new("#{Rails.root}/log/import.log") logger = Rails.logger logger.extend(ActiveSupport::Logger.broadcast(stdout_logger)) @@ -135,6 +136,7 @@ namespace :data do next if time_slot.save logger.warn "[seed_fake] Can't create #{idx + 1}#{(idx + 1).ordinal} time slot for facility (id: #{facility.id}). Errors: #{time_slot.errors.full_messages}" + attention_logger.warn "[import] Can't create #{idx + 1}#{(idx + 1).ordinal} time slot for facility '#{facility.name}' (id: #{facility.id}). Errors: #{time_slot.errors.full_messages}" failed_schedules << facility.id end end @@ -168,6 +170,7 @@ namespace :data do ApplicationRecord.transaction do unless facility.save logger.error "[seed_fake] Failed to create Facility (id: #{facility_attribs["id"]}). Errors: #{facility.errors.full_messages}" + attention_logger.error "[import] Failed to create Facility '#{facility.name}' (id: #{facility_attribs["id"]}). Errors: #{facility.errors.full_messages}" next end From 0e8d9168dc86db471dc8790da7b83c6fb6cdf956 Mon Sep 17 00:00:00 2001 From: Fabio Lima Date: Sat, 28 Jun 2025 14:50:31 -0700 Subject: [PATCH 2/2] Fix facility suitability migration --- ...61125211440_change_facility_suitability.rb | 19 +++++++------------ 1 file changed, 7 insertions(+), 12 deletions(-) diff --git a/db/migrate/20161125211440_change_facility_suitability.rb b/db/migrate/20161125211440_change_facility_suitability.rb index e7db6873..5f10e19d 100644 --- a/db/migrate/20161125211440_change_facility_suitability.rb +++ b/db/migrate/20161125211440_change_facility_suitability.rb @@ -1,21 +1,16 @@ class ChangeFacilitySuitability < ActiveRecord::Migration[4.2] + # Define a local model class that only knows about existing columns + class MigrationFacility < ActiveRecord::Base + self.table_name = 'facilities' + end + def up change_column :facilities, :suitability, :string - Facility.find_each do |f| - if f.suitability == "Children" - f.suitability = "children" - f.save - end - end + MigrationFacility.where(suitability: "Children").update_all(suitability: "children") end def down change_column :facilities, :suitability, :string - Facility.find_each do |f| - if f.suitability == "children" - f.suitability = "Children" - f.save - end - end + MigrationFacility.where(suitability: "children").update_all(suitability: "Children") end end