Merge branch 'main' into strippers_and_rob_both_work_late

* main: Denormalize Status Message to that Entry Look Up Can Be Fast (#913) ⚙️ Adding dry-monads as development dependency
samvera · Feb 8, 2024 · 8da9aa9 · 8da9aa9
2 parents 1adeff0 + 8ce15d9
commit 8da9aa9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/bulkrax.gemspec b/bulkrax.gemspec
@@ -33,6 +33,7 @@ Gem::Specification.new do |s|
   s.add_dependency 'rubyzip'
   s.add_dependency 'simple_form'
 
+  s.add_development_dependency 'dry-monads'
   s.add_development_dependency 'sqlite3', '~> 1.3.13'
   s.add_development_dependency 'simplecov'
   s.add_development_dependency 'redis', '~> 4.2'

diff --git a/db/migrate/20240208005801_denormalize_status_message.rb b/db/migrate/20240208005801_denormalize_status_message.rb
@@ -1,7 +1,7 @@
 class DenormalizeStatusMessage < ActiveRecord::Migration[5.2]
   def change
-    add_column :bulkrax_entries, :status_message, :string, default: 'Pending'
-    add_column :bulkrax_importers, :status_message, :string, default: 'Pending'
-    add_column :bulkrax_exporters, :status_message, :string, default: 'Pending'
+    add_column :bulkrax_entries, :status_message, :string, default: 'Pending' unless column_exists?(:bulkrax_entries, :status_message)
+    add_column :bulkrax_importers, :status_message, :string, default: 'Pending' unless column_exists?(:bulkrax_importers, :status_message)
+    add_column :bulkrax_exporters, :status_message, :string, default: 'Pending' unless column_exists?(:bulkrax_exporters, :status_message)
   end
 end