remove all users from /admin and improve auth method

Author: ptgms

app/admin/dashboard.rb

@@ -14,14 +14,10 @@
     #
     columns do
       column do
-        panel "User" do
-          ul do
-            User.all.map do |user|
-            li link_to(user.email, edit_user_registration_path(user))
-          end
+        panel "Users" do
+          para link_to("Manage all users (#{User.count})", admin_users_path)
         end
       end
-    end
 
     column do
       panel "Info" do

app/controllers/application_controller.rb

@@ -18,6 +18,10 @@ def configure_permitted_parameters
     devise_parameter_sanitizer.permit(:account_update, keys: [:username])
   end
 
+  def authenticate_admin!
+    redirect_to root_path unless current_user&.admin
+  end
+
   def user_not_authorized
     flash[:alert] = 'You are not authorized to perform this action.'
     redirect_to(request.referrer || root_path)

config/initializers/active_admin.rb

@@ -98,7 +98,10 @@
   # will call the method to return the path.
   #
   # Default:
-  config.logout_link_path = :destroy_admin_user_session_path
+  config.logout_link_path = :destroy_user_session_path
+  config.logout_link_method = :delete
+  config.authentication_method = :authenticate_admin!
+  config.current_user_method = :current_user
 
   # This setting changes the http method used when rendering the
   # link. For example :get, :delete, :put, etc..
@@ -290,14 +293,4 @@
   # You can inherit it with own class and inject it for all resources
   #
   # config.order_clause = MyOrderClause
-  def authenticate_admin!
-    redirect_to new_user_session_path unless current_user && current_user.admin
-  end
-
-  ActiveAdmin.setup do |config|
-    config.authentication_method = :authenticate_admin!
-    config.current_user_method = :current_user
-    config.logout_link_path = :destroy_user_session_path
-    config.logout_link_method = :delete
-  end
 end