Merge pull request #27 from ombulabs/refactor

Refactor

Author: rishijain

app/assets/stylesheets/application.css

@@ -21,3 +21,30 @@ th, td {
 th {
   background-color: #f2f2f2;
 }
+.inline-form {
+  display: inline;
+}
+
+/* Reusable button base style */
+.btn {
+  color: white;
+  border: none;
+  padding: 6px 12px;
+  margin-left: 4px;
+  border-radius: 4px;
+  cursor: pointer;
+}
+
+/* Specific button variants */
+.approve-btn {
+  background-color: #4CAF50;
+}
+
+.reject-btn {
+  background-color: #f44336;
+}
+
+.pending-btn {
+  background-color: yellow;
+  color: black;
+}

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

@@ -1,5 +1,7 @@
 class PuzzlesController < ApplicationController
   def index
-    @puzzles = Puzzle.all
+    @pending_puzzles = Puzzle.pending
+    @approved_puzzles = Puzzle.approved
+    @rejected_puzzles = Puzzle.rejected
   end
 end

app/views/puzzles/_puzzles_table.html.erb

@@ -0,0 +1,38 @@
+<table>
+  <thead>
+    <tr>
+      <th>Question</th>
+      <th>Answer</th>
+      <th>Explanation</th>
+      <th>Link</th>
+      <th>Actions</th>
+    </tr>
+  </thead>
+  <tbody>
+    <% puzzles.each do |puzzle| %>
+      <tr>
+        <td><%= puzzle.question %></td>
+        <td><%= puzzle.answer %></td>
+        <td><%= puzzle.explanation %></td>
+        <td>
+          <% if puzzle.link.present? %>
+            <%= link_to 'View', puzzle.link, target: '_blank' %>
+            &nbsp;
+          <% end %>
+        </td>
+        <td>
+          <% if actions == :pending %>
+            <%= button_to 'Approve', puzzle_state_path(puzzle, state: :approved), method: :patch, form_class: 'inline-form', class: 'btn approve-btn' %>
+            <%= button_to 'Reject', puzzle_state_path(puzzle, state: :rejected), method: :patch, form_class: 'inline-form', class: 'btn reject-btn' %>
+          <% elsif actions == :approved %>
+            <%= button_to 'Reject', puzzle_state_path(puzzle, state: :rejected), method: :patch, form_class: 'inline-form', class: 'btn reject-btn' %>
+            <%= button_to 'Pending', puzzle_state_path(puzzle, state: :pending), method: :patch, form_class: 'inline-form', class: 'btn pending-btn' %>
+          <% elsif actions == :rejected %>
+            <%= button_to 'Approve', puzzle_state_path(puzzle, state: :approved), method: :patch, form_class: 'inline-form', class: 'btn approve-btn' %>
+            <%= button_to 'Pending', puzzle_state_path(puzzle, state: :pending), method: :patch, form_class: 'inline-form', class: 'btn pending-btn' %>
+          <% end %>
+        </td>
+      </tr>
+    <% end %>
+  </tbody>
+</table>

app/views/puzzles/index.html.erb

@@ -1,26 +1,14 @@
 <% if session[:user_token] %>
   <%= link_to 'Logout', session_path(1), data: { turbo_method: :delete } %>
-  <h1>Puzzles</h1>
-  <table>
-    <thead>
-      <tr>
-        <th>Question</th>
-        <th>Answer</th>
-        <th>Explanation</th>
-        <th>Link</th>
-      </tr>
-    </thead>
-    <tbody>
-      <% @puzzles.each do |puzzle| %>
-        <tr>
-          <td><%= puzzle.question %></td>
-          <td><%= puzzle.answer %></td>
-          <td><%= puzzle.explanation %></td>
-          <td><%= link_to 'View', puzzle.link, target: '_blank' if puzzle.link.present? %></td>
-        </tr>
-      <% end %>
-    </tbody>
-  </table>
+
+  <h1>Pending Puzzles</h1>
+  <%= render partial: 'puzzles_table', locals: { puzzles: @pending_puzzles, actions: :pending } %>
+
+  <h1>Approved Puzzles</h1>
+  <%= render partial: 'puzzles_table', locals: { puzzles: @approved_puzzles, actions: :approved } %>
+
+  <h1>Rejected Puzzles</h1>
+  <%= render partial: 'puzzles_table', locals: { puzzles: @rejected_puzzles, actions: :rejected } %>
 <% else %>
   <%= link_to 'Login with Google', '/auth/google_oauth2' %>
 <% end %>

config/routes.rb

@@ -1,5 +1,7 @@
 Rails.application.routes.draw do
-  resources :puzzles, only: [:index]
+  resources :puzzles, only: [:index] do
+    resource :state, only: [:update], module: :puzzles
+  end
   resources :sessions, only: [:create, :destroy]
 
   # Define your application routes per the DSL in https://guides.rubyonrails.org/routing.html