CP-11289 - Change external_account_group_id from integer to string

- Migration updates to change data type from integer to string for external_account_group_id
- Updated database schema and annotation comments
- Fixed service parameter type handling
- Updated test expectations to match new string type
- Added storage location field and index for completed documents
- Removed obsolete search_entries table from schema

app/models/account_group.rb

@@ -8,7 +8,7 @@
 #  name                      :string           not null
 #  created_at                :datetime         not null
 #  updated_at                :datetime         not null
-#  external_account_group_id :integer          not null
+#  external_account_group_id :string           not null
 #
 # Indexes
 #

app/models/document_generation_event.rb

@@ -13,7 +13,7 @@
 # Indexes
 #
 #  index_document_generation_events_on_submitter_id                 (submitter_id)
-#  index_document_generation_events_on_submitter_id_and_event_name  (submitter_id,event_name) UNIQUE WHERE ((event_name)::text = ANY ((ARRAY['start'::character varying, 'complete'::character varying])::text[]))
+#  index_document_generation_events_on_submitter_id_and_event_name  (submitter_id,event_name) UNIQUE WHERE ((event_name)::text = ANY (ARRAY[('start'::character varying)::text, ('complete'::character varying)::text]))
 #
 # Foreign Keys
 #

app/models/email_event.rb

@@ -20,7 +20,7 @@
 #
 #  index_email_events_on_account_id_and_event_datetime  (account_id,event_datetime)
 #  index_email_events_on_email                          (email)
-#  index_email_events_on_email_event_types              (email) WHERE ((event_type)::text = ANY ((ARRAY['bounce'::character varying, 'soft_bounce'::character varying, 'complaint'::character varying, 'soft_complaint'::character varying])::text[]))
+#  index_email_events_on_email_event_types              (email) WHERE ((event_type)::text = ANY (ARRAY[('bounce'::character varying)::text, ('soft_bounce'::character varying)::text, ('complaint'::character varying)::text, ('soft_complaint'::character varying)::text]))
 #  index_email_events_on_emailable                      (emailable_type,emailable_id)
 #  index_email_events_on_message_id                     (message_id)
 #

app/services/external_auth_service.rb

@@ -36,7 +36,7 @@ class ExternalAuthService
 
   def find_or_create_user_with_account_group
     account_group = AccountGroup.find_or_create_by_external_id(
-      @params[:account_group][:external_id],
+      @params[:account_group][:external_id]&.to_s,
       name: @params[:account_group][:name]
     )
 

db/schema.rb

@@ -10,9 +10,8 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[8.0].define(version: 2025_09_10_191227) do
+ActiveRecord::Schema[8.0].define(version: 2025_09_30_175543) do
   # These are extensions that must be enabled in order to support this database
-  enable_extension "btree_gin"
   enable_extension "pg_catalog.plpgsql"
 
   create_table "access_tokens", force: :cascade do |t|
@@ -44,7 +43,7 @@ ActiveRecord::Schema[8.0].define(version: 2025_09_10_191227) do
   end
 
   create_table "account_groups", force: :cascade do |t|
-    t.integer "external_account_group_id", null: false
+    t.string "external_account_group_id", null: false
     t.string "name", null: false
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
@@ -115,7 +114,9 @@ ActiveRecord::Schema[8.0].define(version: 2025_09_10_191227) do
     t.string "sha256", null: false
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
+    t.string "storage_location", default: "secured"
     t.index ["sha256"], name: "index_completed_documents_on_sha256"
+    t.index ["storage_location"], name: "index_completed_documents_on_storage_location"
     t.index ["submitter_id"], name: "index_completed_documents_on_submitter_id"
   end
 
@@ -172,7 +173,7 @@ ActiveRecord::Schema[8.0].define(version: 2025_09_10_191227) do
     t.string "event_name", null: false
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
-    t.index ["submitter_id", "event_name"], name: "index_document_generation_events_on_submitter_id_and_event_name", unique: true, where: "((event_name)::text = ANY ((ARRAY['start'::character varying, 'complete'::character varying])::text[]))"
+    t.index ["submitter_id", "event_name"], name: "index_document_generation_events_on_submitter_id_and_event_name", unique: true, where: "((event_name)::text = ANY (ARRAY[('start'::character varying)::text, ('complete'::character varying)::text]))"
     t.index ["submitter_id"], name: "index_document_generation_events_on_submitter_id"
   end
 
@@ -189,7 +190,7 @@ ActiveRecord::Schema[8.0].define(version: 2025_09_10_191227) do
     t.datetime "created_at", null: false
     t.index ["account_id", "event_datetime"], name: "index_email_events_on_account_id_and_event_datetime"
     t.index ["email"], name: "index_email_events_on_email"
-    t.index ["email"], name: "index_email_events_on_email_event_types", where: "((event_type)::text = ANY ((ARRAY['bounce'::character varying, 'soft_bounce'::character varying, 'complaint'::character varying, 'soft_complaint'::character varying])::text[]))"
+    t.index ["email"], name: "index_email_events_on_email_event_types", where: "((event_type)::text = ANY (ARRAY[('bounce'::character varying)::text, ('soft_bounce'::character varying)::text, ('complaint'::character varying)::text, ('soft_complaint'::character varying)::text]))"
     t.index ["emailable_type", "emailable_id"], name: "index_email_events_on_emailable"
     t.index ["message_id"], name: "index_email_events_on_message_id"
   end
@@ -282,19 +283,6 @@ ActiveRecord::Schema[8.0].define(version: 2025_09_10_191227) do
     t.index ["uid"], name: "index_oauth_applications_on_uid", unique: true
   end
 
-  create_table "search_entries", force: :cascade do |t|
-    t.string "record_type", null: false
-    t.bigint "record_id", null: false
-    t.bigint "account_id", null: false
-    t.tsvector "tsvector", null: false
-    t.datetime "created_at", null: false
-    t.datetime "updated_at", null: false
-    t.index ["account_id", "tsvector"], name: "index_search_entries_on_account_id_tsvector_submission", where: "((record_type)::text = 'Submission'::text)", using: :gin
-    t.index ["account_id", "tsvector"], name: "index_search_entries_on_account_id_tsvector_submitter", where: "((record_type)::text = 'Submitter'::text)", using: :gin
-    t.index ["account_id", "tsvector"], name: "index_search_entries_on_account_id_tsvector_template", where: "((record_type)::text = 'Template'::text)", using: :gin
-    t.index ["record_id", "record_type"], name: "index_search_entries_on_record_id_and_record_type", unique: true
-  end
-
   create_table "submission_events", force: :cascade do |t|
     t.integer "submission_id", null: false
     t.integer "submitter_id"

spec/models/account_group_spec.rb

@@ -8,7 +8,7 @@
 #  name                      :string           not null
 #  created_at                :datetime         not null
 #  updated_at                :datetime         not null
-#  external_account_group_id :integer          not null
+#  external_account_group_id :string           not null
 #
 # Indexes
 #

spec/services/external_auth_service_spec.rb

@@ -51,7 +51,7 @@ RSpec.describe ExternalAuthService do
         token = described_class.new(params).authenticate_user
 
         expect(token).to be_present
-        expect(AccountGroup.last.external_account_group_id).to eq(789)
+        expect(AccountGroup.last.external_account_group_id).to eq('789')
         expect(User.last.external_user_id).to eq(123)
       end
     end