Skip to content

Commit

Permalink
Remove unused remember_created_at user column (#10429)
Browse files Browse the repository at this point in the history
* Remove unused remember_created_at user column

changelog: Internal, Database, Remove unused column from users table

* Remove user ignored_columns for dropped field
  • Loading branch information
aduth authored Apr 17, 2024
1 parent c4a0cc0 commit 17bdde4
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 5 deletions.
3 changes: 0 additions & 3 deletions app/models/user.rb
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
# frozen_string_literal: true

class User < ApplicationRecord
# To be dropped in: https://github.com/18F/identity-idp/pull/10429
self.ignored_columns = [:remember_created_at]

include NonNullUuid

include ::NewRelic::Agent::MethodTracer
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
class DropRememberCreatedAtFromUsers < ActiveRecord::Migration[7.1]
def change
safety_assured do
remove_column :users, :remember_created_at, :datetime
end
end
end
3 changes: 1 addition & 2 deletions db/schema.rb
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
#
# It's strongly recommended that you check this file into your version control system.

ActiveRecord::Schema[7.1].define(version: 2024_04_11_163520) do
ActiveRecord::Schema[7.1].define(version: 2024_04_15_125124) do
# These are extensions that must be enabled in order to support this database
enable_extension "citext"
enable_extension "pg_stat_statements"
Expand Down Expand Up @@ -588,7 +588,6 @@
create_table "users", id: :serial, force: :cascade do |t|
t.string "reset_password_token", limit: 255
t.datetime "reset_password_sent_at", precision: nil
t.datetime "remember_created_at", precision: nil
t.datetime "created_at", precision: nil
t.datetime "updated_at", precision: nil
t.datetime "confirmed_at", precision: nil
Expand Down

0 comments on commit 17bdde4

Please sign in to comment.