diff options
author | Bob Van Landuyt <bob@vanlanduyt.co> | 2018-04-27 15:44:23 +0300 |
---|---|---|
committer | Bob Van Landuyt <bob@vanlanduyt.co> | 2018-05-04 14:54:43 +0300 |
commit | 65bea3f7d0bf30b5f9a9b3f94567474d3c8f7cbc (patch) | |
tree | ff3b07de61b3a194de60ca55fbc675e1f1131e37 /db/migrate/20180426102016_add_accepted_term_to_users.rb | |
parent | 3629dc338fbe6c351ce89a94caa4c238965ee33a (diff) |
Add `accepted_terms` to users
This will act as a cache, otherwise we would need to load the
`term_agreements` for a user on each request.
Using this field the result we're interested in is loaded when the
current user is loaded, without causing an extra query.
Diffstat (limited to 'db/migrate/20180426102016_add_accepted_term_to_users.rb')
-rw-r--r-- | db/migrate/20180426102016_add_accepted_term_to_users.rb | 23 |
1 files changed, 23 insertions, 0 deletions
diff --git a/db/migrate/20180426102016_add_accepted_term_to_users.rb b/db/migrate/20180426102016_add_accepted_term_to_users.rb new file mode 100644 index 00000000000..3d446f66214 --- /dev/null +++ b/db/migrate/20180426102016_add_accepted_term_to_users.rb @@ -0,0 +1,23 @@ +# See http://doc.gitlab.com/ce/development/migration_style_guide.html +# for more information on how to write migrations for GitLab. + +class AddAcceptedTermToUsers < ActiveRecord::Migration + include Gitlab::Database::MigrationHelpers + + DOWNTIME = false + + disable_ddl_transaction! + + def up + change_table :users do |t| + t.references :accepted_term, + null: true + end + add_concurrent_foreign_key :users, :application_setting_terms, column: :accepted_term_id + end + + def down + remove_foreign_key :users, column: :accepted_term_id + remove_column :users, :accepted_term_id + end +end |