missing records, making migration only add new users

mdeiters · mdeiters · commit 99c17ad4ccd4 · 2016-02-19T11:06:35.000-08:00
diff --git a/db/migrate/20160219190140_add_banned_users_for_migration.rb b/db/migrate/20160219190140_add_banned_users_for_migration.rb
@@ -0,0 +1,5 @@
+class AddBannedUsersForMigration < ActiveRecord::Migration
+  def change
+    add_column :users, :banned_at, :datetime
+  end
+end
diff --git a/db/schema.rb b/db/schema.rb
@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20160219071138) do
+ActiveRecord::Schema.define(version: 20160219190140) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -103,6 +103,7 @@
     t.string   "twitter"
     t.string   "color",                             default: "#111"
     t.integer  "karma",                             default: 1
+    t.datetime "banned_at"
   end
 
   add_index "users", ["email"], name: "index_users_on_email", using: :btree
diff --git a/lib/tasks/db.rake b/lib/tasks/db.rake
@@ -183,12 +183,9 @@ namespace :db do
               user.company = team[:name]
             end
 
-            if row[:banned_at].nil?
-              Rails.logger.info "#{row[:username]} => #{row[:email]}"
-              user.save!
-            else
-              Rails.logger.info "skipping banned user #{row[:username]}"
-            end
+
+            Rails.logger.info "#{row[:username]} => #{row[:email]}"
+            user.save!            
           end
         end
       end
