Skip to content

Commit

Permalink
DEV: Drop old tables that have been renamed (#221)
Browse files Browse the repository at this point in the history
It is now safe to drop the old table.

The new tables are prefixed with topic-voting-, the old tables are discourse-voting-.

This commit also transfers the sequences to the new tables.
  • Loading branch information
nattsw authored Dec 3, 2024
1 parent 52349b9 commit 6d52b8c
Show file tree
Hide file tree
Showing 2 changed files with 35 additions and 0 deletions.
22 changes: 22 additions & 0 deletions db/migrate/20241203125415_rename_reassign_sequences.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
# frozen_string_literal: true

class RenameReassignSequences < ActiveRecord::Migration[7.0]
def up
reassign_sequence("discourse_voting_topic_vote_count_id_seq", "topic_voting_topic_vote_count")
reassign_sequence("discourse_voting_votes_id_seq", "topic_voting_votes")
reassign_sequence("discourse_voting_category_settings_id_seq", "topic_voting_category_settings")
end

def down
raise ActiveRecord::IrreversibleMigration
end

private

def reassign_sequence(sequence_name, new_table_name)
execute <<~SQL
ALTER SEQUENCE #{sequence_name}
OWNED BY #{new_table_name}.id;
SQL
end
end
13 changes: 13 additions & 0 deletions db/post_migrate/20241203125523_drop_old_discourse_voting_tables.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
# frozen_string_literal: true

class DropOldDiscourseVotingTables < ActiveRecord::Migration[7.0]
def up
drop_table :discourse_voting_topic_vote_count, if_exists: true
drop_table :discourse_voting_votes, if_exists: true
drop_table :discourse_voting_category_settings, if_exists: true
end

def down
raise ActiveRecord::IrreversibleMigration
end
end

0 comments on commit 6d52b8c

Please sign in to comment.