Skip to content
This repository was archived by the owner on Jul 22, 2025. It is now read-only.

Commit c991184

Browse files
committed
FIX: Set default for existing records.
We'll later copy the correct value from content_range. 1 should be the min highest post number a topic has.
1 parent 46fcdb6 commit c991184

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

db/migrate/20250115173456_add_highest_target_number_to_ai_summary.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
# frozen_string_literal: true
22
class AddHighestTargetNumberToAiSummary < ActiveRecord::Migration[7.2]
33
def up
4-
add_column :ai_summaries, :highest_target_number, :integer, null: false
4+
add_column :ai_summaries, :highest_target_number, :integer, null: false, default: 1
55

66
execute <<~SQL
7-
UPDATE ai_summaries SET highest_target_number = GREATEST(UPPER(content_range) - 1, 1)
7+
UPDATE ai_summaries SET highest_target_number = CASE WHEN content_range IS NOT NULLGREATEST(UPPER(content_range) - 1, 1)
88
SQL
99
end
1010

0 commit comments

Comments
 (0)