Removing quickpost (#461)

* intial commit

* edited migration

* Fixed Lint

* added permissions to migration

* Fixed Lint

* Updated migration

app/jobs/user_archive_job.rb

@@ -61,7 +61,6 @@ def keep_entities # rubocop:disable Metrics/MethodLength
      PhotoAlbum,
      PhotoTag,
      Poll,
-     QuickpostMessage,
      Form::Form,
      Form::Response,
      Forum::Post,

config/routes.rb

@@ -73,7 +73,6 @@
     end
     get 'users/me/nextcloud', to: 'users#nextcloud'
 
-    jsonapi_resources :quickpost_messages
     jsonapi_resources :vacancies
 
     namespace :debit do

db/migrate/20241113091607_remove_quickpost.rb

@@ -0,0 +1,10 @@
+class RemoveQuickpost < ActiveRecord::Migration[7.0]
+  def change
+    drop_table :quickpost_messages
+
+    Permission.find_by(name: 'quickpost_message.create')&.destroy
+    Permission.find_by(name: 'quickpost_message.read')&.destroy
+    Permission.find_by(name: 'quickpost_message.update')&.destroy
+    Permission.find_by(name: 'quickpost_message.destroy')&.destroy
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[7.0].define(version: 2024_10_27_103012) do
+ActiveRecord::Schema[7.0].define(version: 2024_11_13_091607) do
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
 
@@ -474,18 +474,7 @@
     t.index ["author_id"], name: "index_polls_on_author_id"
     t.index ["form_id"], name: "index_polls_on_form_id"
   end
-
-  create_table "quickpost_messages", id: :serial, force: :cascade do |t|
-    t.integer "author_id"
-    t.text "message"
-    t.datetime "created_at", precision: nil, null: false
-    t.datetime "updated_at", precision: nil, null: false
-    t.datetime "deleted_at", precision: nil
-    t.datetime "datetime", precision: nil
-    t.index ["datetime"], name: "index_quickpost_messages_on_datetime"
-    t.index ["deleted_at"], name: "index_quickpost_messages_on_deleted_at"
-  end
-
+3
   create_table "room_adverts", force: :cascade do |t|
     t.string "house_name", null: false
     t.string "contact", null: false

db/seeds/content.rb

@@ -45,10 +45,6 @@
   end
 end
 
-members.sample(80).each { |user| FactoryBot.create(:quickpost_message, author: user) }
-# To ensure at least one user creating (more than) two quickpost messages
-members.sample(30).each { |user| FactoryBot.create(:quickpost_message, author: user) }
-
 members.sample(4).each do |user|
   FactoryBot.create(:activity, author: user)
 end

db/seeds/permissions.rb

@@ -34,7 +34,6 @@ def create_permissions(permission_map)
   'photo' => %i[create read update destroy],
   'photo_comment' => %i[create read update destroy],
   'photo_tag' => %i[create read update destroy],
-  'quickpost_message' => %i[create read update destroy],
   'room_advert' => %i[create read update destroy],
   'stored_mail' => %i[read destroy],
   'forum/category' => %i[create read update destroy],
@@ -76,7 +75,6 @@ def create_permissions(permission_map)
   'photo' => %i[create read],
   'photo_comment' => %i[create read],
   'photo_tag' => %i[create read],
-  'quickpost_message' => %i[create read],
   'room_advert' => %i[create read],
   'forum/category' => %i[read],
   'forum/thread' => %i[create read],
@@ -111,7 +109,6 @@ def create_permissions(permission_map)
   'photo' => %i[read],
   'photo_comment' => %i[create read],
   'photo_tag' => %i[create read],
-  'quickpost_message' => %i[create read],
   'room_advert' => %i[],
   'forum/category' => %i[read],
   'forum/thread' => %i[create read],

spec/jobs/user_archive_job_spec.rb

@@ -17,7 +17,6 @@
         create(:photo, uploader: user)
         create(:photo_album, author: user)
         create(:poll, author: user)
-        create(:quickpost_message, author: user)
         create(:response, user:)
         create(:form, author: user)
         create(:post, author: user)
@@ -33,7 +32,6 @@
       it { expect { job }.to change { Photo.last.uploader }.from(user).to(archive_user) }
       it { expect { job }.to change { PhotoAlbum.last.author }.from(user).to(archive_user) }
       it { expect { job }.to change { Poll.last.author }.from(user).to(archive_user) }
-      it { expect { job }.to change { QuickpostMessage.last.author }.from(user).to(archive_user) }
       it { expect { job }.to change { Form::Response.last.user }.from(user).to(archive_user) }
       it { expect { job }.to change { Form::Form.last.author }.from(user).to(archive_user) }
       it { expect { job }.to change { Forum::Post.last.author }.from(user).to(archive_user) }