Skip to content

Commit decbe79

Browse files
authored
repair migrations (#1048)
**What** This makes a couple of updates to the migrations based on [this conversation][is]: * Rename `Api` -> `API`. This is expected by Rails inflections. * Add conditional checks to migrations that look for a user. If a person hasn't set up an account yet, the migrations should still run gracefully. [is]: #1046
1 parent 2e78282 commit decbe79

4 files changed

+7
-5
lines changed

db/migrate/20130522014405_add_api_key_to_user.rb

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# frozen_string_literal: true
22

3-
class AddApiKeyToUser < ActiveRecord::Migration[4.2]
3+
class AddAPIKeyToUser < ActiveRecord::Migration[4.2]
44
def change
55
add_column :users, :api_key, :string
66
end

db/migrate/20230223045525_add_null_false_to_associations.rb

+4-2
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,10 @@
22

33
class AddNullFalseToAssociations < ActiveRecord::Migration[7.0]
44
def change
5-
Feed.where(user: nil).update_all(user_id: User.first.id)
6-
Group.where(user: nil).update_all(user_id: User.first.id)
5+
if User.any?
6+
Feed.where(user: nil).update_all(user_id: User.first.id)
7+
Group.where(user: nil).update_all(user_id: User.first.id)
8+
end
79

810
change_column_null :feeds, :user_id, false
911
change_column_null :groups, :user_id, false

db/migrate/20230223231930_add_username_to_users.rb

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ class AddUsernameToUsers < ActiveRecord::Migration[7.0]
44
def change
55
add_column :users, :username, :string
66
add_index :users, :username, unique: true
7-
User.first.update!(username: "stringer")
7+
User.first.update!(username: "stringer") if User.any?
88
change_column_null :users, :username, false
99
end
1010
end

db/migrate/20230313034938_add_admin_to_users.rb

+1-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ class AddAdminToUsers < ActiveRecord::Migration[7.0]
44
def change
55
add_column :users, :admin, :boolean
66

7-
User.first.update!(admin: true)
7+
User.first.update!(admin: true) if User.any?
88

99
change_column_null :users, :admin, false
1010
end

0 commit comments

Comments
 (0)