Delete beta access

seuros · seuros · commit 2e2dad23f365 · 2014-07-19T16:08:01.000Z
diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb
@@ -201,10 +201,6 @@ def is_admin?
     signed_in? && current_user.admin?
   end
 
-  def require_beta_access!
-    return head(:forbidden) unless signed_in? && current_user.has_beta_access?
-  end
-
   def iphone_user_agent?
     request.env["HTTP_USER_AGENT"] && request.env["HTTP_USER_AGENT"][/(Mobile\/.+Safari)/]
   end
diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
@@ -66,12 +66,6 @@ def index
     end
   end
 
-  def beta
-    return head(:forbidden) unless signed_in? && (current_user.beta_access? || current_user.admin?)
-    @user  = current_user
-    @users = User.where('banner is not null')
-  end
-
   def create
     @user = User.for_omniauth(oauth)
     Rails.logger.debug("Creating User: #{@user.inspect}")  if ENV['DEBUG']
diff --git a/app/models/user.rb b/app/models/user.rb
@@ -446,10 +446,6 @@ def total_achievements
     badges_count
   end
 
-  def has_beta_access?
-    admin? || beta_access
-  end
-
   def award(badge)
     new_badge = self.badges.of_type(badge).first || self.badges.build(badge_class_name: badge.class.name)
   end
@@ -1366,7 +1362,6 @@ def manage_github_orgs
 #  linkedin_secret               :string(255)
 #  last_email_sent               :datetime
 #  linkedin_public_url           :string(255)
-#  beta_access                   :boolean          default(FALSE)
 #  redemptions                   :text
 #  endorsements_count            :integer          default(0)
 #  team_document_id              :string(255)      indexed
diff --git a/app/views/users/beta.html.haml b/app/views/users/beta.html.haml
diff --git a/config/initializers/airbrake.rb b/config/initializers/airbrake.rb
@@ -6,5 +6,7 @@
     config.secure  = config.port == 443
   end
 else
-  Rails.logger.warn '[WTF WARNING] Someone deleted airbrake and forgot the initializer'
+  unless Rails.env.test? || Rails.env.development?
+    Rails.logger.warn '[WTF WARNING] Someone deleted airbrake and forgot the initializer'
+  end
 end
diff --git a/config/routes.rb b/config/routes.rb
@@ -247,7 +247,6 @@
 #                             PUT      /users/:id(.:format)                                   users#update
 #                             DELETE   /users/:id(.:format)                                   users#destroy
 #              clear_provider GET      /clear/:id/:provider(.:format)                         users#clear_provider
-#                      visual GET      /visual(.:format)                                      users#beta
 #                     refresh GET      /refresh/:username(.:format)                           users#refresh
 #       random_accomplishment GET      /nextaccomplishment(.:format)                          highlights#random
 #                   add_skill GET      /add-skill(.:format)                                   skills#create
@@ -467,7 +466,6 @@
   end
 
   get '/clear/:id/:provider' => 'users#clear_provider', as: :clear_provider
-  get '/visual' => 'users#beta'
   get '/refresh/:username' => 'users#refresh', as: :refresh
   get '/nextaccomplishment' => 'highlights#random', as: :random_accomplishment
   get '/add-skill' => 'skills#create', as: :add_skill, :via => :post
diff --git a/db/migrate/20140719160422_remove_beta_access.rb b/db/migrate/20140719160422_remove_beta_access.rb
@@ -0,0 +1,5 @@
+class RemoveBetaAccess < ActiveRecord::Migration
+  def up
+    remove_column :users, :beta_access
+  end
+end
diff --git a/spec/fabricators/user_fabricator.rb b/spec/fabricators/user_fabricator.rb
@@ -74,7 +74,6 @@
 #  linkedin_secret               :string(255)
 #  last_email_sent               :datetime
 #  linkedin_public_url           :string(255)
-#  beta_access                   :boolean          default(FALSE)
 #  redemptions                   :text
 #  endorsements_count            :integer          default(0)
 #  team_document_id              :string(255)      indexed

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +class RemoveBetaAccess < ActiveRecord::Migration
 +  def up
 +    remove_column :users, :beta_access
 +  end
 +end