created a puzzles states controller to handle puzzle states

Author: rishijain

app/controllers/puzzles/states_controller.rb

@@ -0,0 +1,12 @@
+module Puzzles
+  class StatesController < ApplicationController
+    def update
+      puzzle = Puzzle.find(params[:puzzle_id])
+      if puzzle.update(state: params[:state])
+        redirect_to puzzles_path, notice: "Puzzle state updated to #{puzzle.state}."
+      else
+        redirect_to puzzles_path, alert: "Failed to update puzzle state."
+      end
+    end
+  end
+end

app/controllers/puzzles_controller.rb

@@ -4,22 +4,4 @@ def index
     @approved_puzzles = Puzzle.approved
     @rejected_puzzles = Puzzle.rejected
   end
-
-  def approve
-    puzzle = Puzzle.find(params[:id])
-    puzzle.update(state: :approved)
-    redirect_to puzzles_path, notice: "Puzzle approved."
-  end
-
-  def reject
-    puzzle = Puzzle.find(params[:id])
-    puzzle.update(state: :rejected)
-    redirect_to puzzles_path, notice: "Puzzle rejected."
-  end
-
-  def pending
-    puzzle = Puzzle.find(params[:id])
-    puzzle.update(state: :pending)
-    redirect_to puzzles_path, notice: "Puzzle marked as pending."
-  end
 end

app/views/puzzles/index.html.erb

@@ -24,8 +24,8 @@
             <% end %>
           </td>
           <td>
-            <%= button_to 'Approve', approve_puzzle_path(puzzle), method: :patch, form_class: 'inline-form', class: 'approve-btn' %>
-            <%= button_to 'Reject', reject_puzzle_path(puzzle), method: :patch, form_class: 'inline-form', class: 'reject-btn' %>
+            <%= button_to 'Approve', puzzle_state_path(puzzle, state: :approved), method: :patch, form_class: 'inline-form', class: 'approve-btn' %>
+            <%= button_to 'Reject', puzzle_state_path(puzzle, state: :rejected), method: :patch, form_class: 'inline-form', class: 'reject-btn' %>
           </td>
         </tr>
       <% end %>
@@ -56,8 +56,8 @@
             <% end %>
           </td>
           <td>
-            <%= button_to 'Reject', reject_puzzle_path(puzzle), method: :patch, form_class: 'inline-form', class: 'reject-btn' %>
-            <%= button_to 'Pending', pending_puzzle_path(puzzle), method: :patch, form_class: 'inline-form', class: 'pending-btn' %>
+            <%= button_to 'Reject', puzzle_state_path(puzzle, state: :rejected), method: :patch, form_class: 'inline-form', class: 'reject-btn' %>
+            <%= button_to 'Pending', puzzle_state_path(puzzle, state: :pending), method: :patch, form_class: 'inline-form', class: 'pending-btn' %>
           </td>
         </tr>
       <% end %>
@@ -88,14 +88,13 @@
             <% end %>
           </td>
           <td>
-            <%= button_to 'Approve', approve_puzzle_path(puzzle), method: :patch, form_class: 'inline-form', class: 'approve-btn' %>
-            <%= button_to 'Pending', pending_puzzle_path(puzzle), method: :patch, form_class: 'inline-form', class: 'pending-btn' %>
+            <%= button_to 'Approve', puzzle_state_path(puzzle, state: :approved), method: :patch, form_class: 'inline-form', class: 'approve-btn' %>
+            <%= button_to 'Pending', puzzle_state_path(puzzle, state: :pending), method: :patch, form_class: 'inline-form', class: 'pending-btn' %>
           </td>
         </tr>
       <% end %>
     </tbody>
   </table>
-
 <% else %>
   <%= link_to 'Login with Google', '/auth/google_oauth2' %>
 <% end %>

config/routes.rb

@@ -1,10 +1,6 @@
 Rails.application.routes.draw do
   resources :puzzles, only: [:index] do
-    member do
-      patch :approve
-      patch :reject
-      patch :pending
-    end
+    resource :state, only: [:update], module: :puzzles
   end
   resources :sessions, only: [:create, :destroy]