Remove unused show action for notices

riseuplabs · Oct 18, 2018 · eccd924 · eccd924
1 parent e1f2657
commit eccd924
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 10 deletions.
diff --git a/app/controllers/me/notices_controller.rb b/app/controllers/me/notices_controller.rb
@@ -1,5 +1,5 @@
 class Me::NoticesController < Me::BaseController
-  before_filter :fetch_notice, only: %i[show destroy]
+  before_filter :fetch_notice, only: %i[destroy]
 
   def index
     @notices = Notice.for_user(current_user)
@@ -11,14 +11,6 @@ def index
     @pages = Page.paginate_by_path '', options_for_me, pagination_params
   end
 
-  def show
-    url = send(@notice.redirect_path, @notice.redirect_object)
-    respond_to do |format|
-      format.html { redirect_to url }
-      format.js { render(:update) { |page| page.redirect_to url } }
-    end
-  end
-
   #
   # don't actually destroy, just mark dismissed
   #

diff --git a/config/routes.rb b/config/routes.rb
@@ -58,7 +58,7 @@
     delete 'notices/destroy_all',
       to: 'notices#destroy_all',
       as: 'notices_destroy_all'
-    resources :notices, only: %i[index show destroy]
+    resources :notices, only: %i[index destroy]
     get '', to: 'notices#index', as: 'home'
     # resource  :page, only: [:new, :create]
     resources :recent_pages, only: [:index]