From 26dfb8a489fad26196e1ce8a29acd2c3623bda50 Mon Sep 17 00:00:00 2001 From: Alan Guo Xiang Tan Date: Fri, 8 Sep 2023 12:41:56 +0800 Subject: [PATCH] DEV: Remove chat related migration (#23449) What motivated this change? A core migration contains chat related code and this should not be the case since chat related migration code should live in the chat plugin. What does this change do? This change removes the migration which was introduced to keep existing sites on the legacy navigation menu as well as keep chat disabled when the defaults for the `navigation_menu` and `chat_enabled` site settings were flipped. Since this migration doesn't apply to new sites and the migration has already been introduced for 9 months, it is safe for us to remove it. --- .../20221212225921_enable_sidebar_and_chat.rb | 23 ------------------- 1 file changed, 23 deletions(-) delete mode 100644 db/migrate/20221212225921_enable_sidebar_and_chat.rb diff --git a/db/migrate/20221212225921_enable_sidebar_and_chat.rb b/db/migrate/20221212225921_enable_sidebar_and_chat.rb deleted file mode 100644 index 32c8b5f3497..00000000000 --- a/db/migrate/20221212225921_enable_sidebar_and_chat.rb +++ /dev/null @@ -1,23 +0,0 @@ -# frozen_string_literal: true - -class EnableSidebarAndChat < ActiveRecord::Migration[7.0] - def up - # keep sidebar legacy and chat disabled for existing sites - if Migration::Helpers.existing_site? - execute <<~SQL - INSERT INTO site_settings(name, data_type, value, created_at, updated_at) - VALUES('chat_enabled', 5, 'f', NOW(), NOW()) - ON CONFLICT (name) DO NOTHING - SQL - execute <<~SQL - INSERT INTO site_settings(name, data_type, value, created_at, updated_at) - VALUES('navigation_menu', 7, 'legacy', NOW(), NOW()) - ON CONFLICT (name) DO NOTHING - SQL - end - end - - def down - raise ActiveRecord::IrreversibleMigration - end -end