Skip to content

Commit 5abf80c

Browse files
authored
FIX: do not mark column read only so certain deployments work (#663)
In some case we may be deploying migrations, seeding and then running post migrations, we need this to work so we give up on this small window of protection
1 parent 52a7dd2 commit 5abf80c

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

db/migrate/20240609061418_tool_details_and_command_removal.rb

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,10 @@ class ToolDetailsAndCommandRemoval < ActiveRecord::Migration[7.0]
44
def change
55
add_column :ai_personas, :tool_details, :boolean, default: true, null: false
66
add_column :ai_personas, :tools, :json, null: false, default: []
7-
Migration::ColumnDropper.mark_readonly(:ai_personas, :commands)
7+
# we can not do this cause we are seeding the data and in certain
8+
# build scenarios we seed prior to running post migrations
9+
# this risks potentially dropping data but the window is small
10+
# Migration::ColumnDropper.mark_readonly(:ai_personas, :commands)
811

912
execute <<~SQL
1013
UPDATE ai_personas

0 commit comments

Comments
 (0)