diff --git a/app/models/account_group.rb b/app/models/account_group.rb index e2bde3f0..02aa57ba 100644 --- a/app/models/account_group.rb +++ b/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 # diff --git a/app/models/document_generation_event.rb b/app/models/document_generation_event.rb index f4e0a701..24d71ba3 100644 --- a/app/models/document_generation_event.rb +++ b/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 # diff --git a/app/models/email_event.rb b/app/models/email_event.rb index 3c6aac61..3aaa5db9 100644 --- a/app/models/email_event.rb +++ b/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) # diff --git a/app/services/external_auth_service.rb b/app/services/external_auth_service.rb index 569f85ca..e72b5912 100644 --- a/app/services/external_auth_service.rb +++ b/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] ) diff --git a/db/schema.rb b/db/schema.rb index 626e9dac..a95e2555 100644 --- a/db/schema.rb +++ b/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" diff --git a/spec/models/account_group_spec.rb b/spec/models/account_group_spec.rb index bc291c26..d2c02f8e 100644 --- a/spec/models/account_group_spec.rb +++ b/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 # diff --git a/spec/services/external_auth_service_spec.rb b/spec/services/external_auth_service_spec.rb index b1422177..c2dca615 100644 --- a/spec/services/external_auth_service_spec.rb +++ b/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