Update migrations to utilize different data structure

Author: edwinthinks

db/migrate/20201230233349_create_partner_group_memberships.rb

@@ -0,0 +1,7 @@
+class AddUniqunessIdxOnNameAndOrganizationIdInPartnerGroups < ActiveRecord::Migration[6.1]
+  disable_ddl_transaction!
+
+  def change
+    add_index :partner_groups, [:name, :organization_id], unique: true, algorithm: :concurrently
+  end
+end

db/migrate/20210116200129_create_partner_group_items.rb

@@ -0,0 +1,7 @@
+class AddReferenceBetweenPartnersAndPartnerGroups < ActiveRecord::Migration[6.1]
+  disable_ddl_transaction!
+
+  def change
+    add_reference :partners, :partner_groups, index: { algorithm: :concurrently }
+  end
+end

db/migrate/20210729003044_add_uniquness_idx_on_name_and_organization_id_in_partner_groups.rb

@@ -0,0 +1,10 @@
+class AddJoinTableBetweenPartnerGroupsToItemCategories < ActiveRecord::Migration[6.1]
+  def change
+    create_table :partner_groups_item_categories do |t|
+      t.references :partner, foreign_key: true
+      t.references :item_categories, foreign_key: true
+
+      t.timestamps
+    end
+  end
+end

db/migrate/20210729003314_add_reference_between_partners_and_partner_groups.rb

@@ -10,8 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20_210_107_175_100) do
-
+ActiveRecord::Schema.define(version: 20_210_729_003_516) do
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
 
@@ -33,7 +32,7 @@
     t.bigint "record_id", null: false
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
-    t.index ["record_type", "record_id", "name"], name: "index_action_text_rich_texts_uniqueness", unique: true
+    t.index %w(record_type record_id name), name: "index_action_text_rich_texts_uniqueness", unique: true
   end
 
   create_table "active_storage_attachments", force: :cascade do |t|
@@ -43,7 +42,7 @@
     t.bigint "blob_id", null: false
     t.datetime "created_at", null: false
     t.index ["blob_id"], name: "index_active_storage_attachments_on_blob_id"
-    t.index ["record_type", "record_id", "name", "blob_id"], name: "index_active_storage_attachments_uniqueness", unique: true
+    t.index %w(record_type record_id name blob_id), name: "index_active_storage_attachments_uniqueness", unique: true
   end
 
   create_table "active_storage_blobs", force: :cascade do |t|
@@ -63,7 +62,7 @@
     t.string "variation_digest", null: false
     t.datetime "created_at", precision: 6, null: false
     t.datetime "updated_at", precision: 6, null: false
-    t.index ["blob_id", "variation_digest"], name: "index_active_storage_variant_records_uniqueness", unique: true
+    t.index %w(blob_id variation_digest), name: "index_active_storage_variant_records_uniqueness", unique: true
   end
 
   create_table "adjustments", id: :serial, force: :cascade do |t|
@@ -100,7 +99,7 @@
     t.datetime "updated_at", null: false
     t.integer "organization_id"
     t.string "barcodeable_type", default: "Item"
-    t.index ["barcodeable_type", "barcodeable_id"], name: "index_barcode_items_on_barcodeable_type_and_barcodeable_id"
+    t.index %w(barcodeable_type barcodeable_id), name: "index_barcode_items_on_barcodeable_type_and_barcodeable_id"
     t.index ["organization_id"], name: "index_barcode_items_on_organization_id"
   end
 
@@ -137,7 +136,7 @@
     t.string "business_name"
     t.float "latitude"
     t.float "longitude"
-    t.index ["latitude", "longitude"], name: "index_diaper_drive_participants_on_latitude_and_longitude"
+    t.index %w(latitude longitude), name: "index_diaper_drive_participants_on_latitude_and_longitude"
   end
 
   create_table "diaper_drives", force: :cascade do |t|
@@ -176,7 +175,7 @@
     t.integer "organization_id"
     t.float "latitude"
     t.float "longitude"
-    t.index ["latitude", "longitude"], name: "index_donation_sites_on_latitude_and_longitude"
+    t.index %w(latitude longitude), name: "index_donation_sites_on_latitude_and_longitude"
     t.index ["organization_id"], name: "index_donation_sites_on_organization_id"
   end
 
@@ -213,7 +212,7 @@
     t.string "value"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
-    t.index ["feature_key", "key", "value"], name: "index_flipper_gates_on_feature_key_and_key_and_value", unique: true
+    t.index %w(feature_key key value), name: "index_flipper_gates_on_feature_key_and_key_and_value", unique: true
   end
 
   create_table "inventory_items", id: :serial, force: :cascade do |t|
@@ -225,12 +224,12 @@
   end
 
   create_table "item_categories", force: :cascade do |t|
-    t.string "name"
+    t.string "name", null: false
     t.text "description"
     t.integer "organization_id", null: false
     t.datetime "created_at", precision: 6, null: false
     t.datetime "updated_at", precision: 6, null: false
-    t.index ["name", "organization_id"], name: "index_item_categories_on_name_and_organization_id", unique: true
+    t.index %w(name organization_id), name: "index_item_categories_on_name_and_organization_id", unique: true
   end
 
   create_table "items", id: :serial, force: :cascade do |t|
@@ -264,7 +263,7 @@
     t.boolean "active", default: true
     t.boolean "visible_to_partners", default: true, null: false
     t.integer "value_in_cents", default: 0
-    t.index ["name", "organization_id"], name: "index_kits_on_name_and_organization_id", unique: true
+    t.index %w(name organization_id), name: "index_kits_on_name_and_organization_id", unique: true
     t.index ["organization_id"], name: "index_kits_on_organization_id"
   end
 
@@ -275,7 +274,7 @@
     t.string "itemizable_type"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
-    t.index ["itemizable_id", "itemizable_type"], name: "index_line_items_on_itemizable_id_and_itemizable_type"
+    t.index %w(itemizable_id itemizable_type), name: "index_line_items_on_itemizable_id_and_itemizable_type"
   end
 
   create_table "manufacturers", force: :cascade do |t|
@@ -306,34 +305,26 @@
     t.integer "default_storage_location"
     t.text "partner_form_fields", default: [], array: true
     t.integer "account_request_id"
-    t.index ["latitude", "longitude"], name: "index_organizations_on_latitude_and_longitude"
+    t.index %w(latitude longitude), name: "index_organizations_on_latitude_and_longitude"
     t.index ["short_name"], name: "index_organizations_on_short_name"
   end
 
-  create_table "partner_group_items", force: :cascade do |t|
-    t.bigint "partner_group_id", null: false
-    t.bigint "item_id", null: false
+  create_table "partner_groups", force: :cascade do |t|
+    t.bigint "organization_id"
+    t.string "name"
     t.datetime "created_at", precision: 6, null: false
     t.datetime "updated_at", precision: 6, null: false
-    t.index ["item_id"], name: "index_partner_group_items_on_item_id"
-    t.index ["partner_group_id"], name: "index_partner_group_items_on_partner_group_id"
+    t.index %w(name organization_id), name: "index_partner_groups_on_name_and_organization_id", unique: true
+    t.index ["organization_id"], name: "index_partner_groups_on_organization_id"
   end
 
-  create_table "partner_group_memberships", force: :cascade do |t|
-    t.bigint "partner_group_id"
+  create_table "partner_groups_item_categories", force: :cascade do |t|
     t.bigint "partner_id"
+    t.bigint "item_categories_id"
     t.datetime "created_at", precision: 6, null: false
     t.datetime "updated_at", precision: 6, null: false
-    t.index ["partner_group_id"], name: "index_partner_group_memberships_on_partner_group_id"
-    t.index ["partner_id"], name: "index_partner_group_memberships_on_partner_id"
-  end
-
-  create_table "partner_groups", force: :cascade do |t|
-    t.bigint "organization_id"
-    t.string "name"
-    t.datetime "created_at", precision: 6, null: false
-    t.datetime "updated_at", precision: 6, null: false
-    t.index ["organization_id"], name: "index_partner_groups_on_organization_id"
+    t.index ["item_categories_id"], name: "index_partner_groups_item_categories_on_item_categories_id"
+    t.index ["partner_id"], name: "index_partner_groups_item_categories_on_partner_id"
   end
 
   create_table "partners", id: :serial, force: :cascade do |t|
@@ -346,7 +337,9 @@
     t.boolean "send_reminders", default: false, null: false
     t.text "notes"
     t.integer "quota"
+    t.bigint "partner_groups_id"
     t.index ["organization_id"], name: "index_partners_on_organization_id"
+    t.index ["partner_groups_id"], name: "index_partners_on_partner_groups_id"
   end
 
   create_table "purchases", force: :cascade do |t|
@@ -391,7 +384,7 @@
     t.float "longitude"
     t.integer "square_footage"
     t.string "warehouse_type"
-    t.index ["latitude", "longitude"], name: "index_storage_locations_on_latitude_and_longitude"
+    t.index %w(latitude longitude), name: "index_storage_locations_on_latitude_and_longitude"
     t.index ["organization_id"], name: "index_storage_locations_on_organization_id"
   end
 
@@ -437,7 +430,7 @@
     t.index ["invitation_token"], name: "index_users_on_invitation_token", unique: true
     t.index ["invitations_count"], name: "index_users_on_invitations_count"
     t.index ["invited_by_id"], name: "index_users_on_invited_by_id"
-    t.index ["invited_by_type", "invited_by_id"], name: "index_users_on_invited_by_type_and_invited_by_id"
+    t.index %w(invited_by_type invited_by_id), name: "index_users_on_invited_by"
     t.index ["reset_password_token"], name: "index_users_on_reset_password_token", unique: true
   end
 
@@ -453,7 +446,7 @@
     t.float "longitude"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
-    t.index ["latitude", "longitude"], name: "index_vendors_on_latitude_and_longitude"
+    t.index %w(latitude longitude), name: "index_vendors_on_latitude_and_longitude"
   end
 
   create_table "versions", force: :cascade do |t|
@@ -483,11 +476,9 @@
   add_foreign_key "kits", "organizations"
   add_foreign_key "manufacturers", "organizations"
   add_foreign_key "organizations", "account_requests"
-  add_foreign_key "partner_group_items", "items"
-  add_foreign_key "partner_group_items", "partner_groups"
-  add_foreign_key "partner_group_memberships", "partner_groups"
-  add_foreign_key "partner_group_memberships", "partners"
   add_foreign_key "partner_groups", "organizations"
+  add_foreign_key "partner_groups_item_categories", "item_categories", column: "item_categories_id"
+  add_foreign_key "partner_groups_item_categories", "partners"
   add_foreign_key "requests", "distributions"
   add_foreign_key "requests", "organizations"
   add_foreign_key "requests", "partners"