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

Commit 4784e7f

Browse files
authored
FIX: Set default for existing records. (#1073)
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 4784e7f

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

app/models/ai_summary.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ def outdated
6464
# updated_at :datetime not null
6565
# summary_type :integer default("complete"), not null
6666
# origin :integer
67-
# highest_target_number :integer not null
67+
# highest_target_number :integer default(1), not null
6868
#
6969
# Indexes
7070
#

db/migrate/20250115173456_add_highest_target_number_to_ai_summary.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
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
77
UPDATE ai_summaries SET highest_target_number = GREATEST(UPPER(content_range) - 1, 1)

0 commit comments

Comments
 (0)