Merge pull request #5 from tamu-edu-students/revert-4-admin-managemen…

…t-display-page Revert "Admin management display page"
tamu-edu-students · Oct 31, 2024 · 7a95457 · 7a95457
2 parents bfbc2a1 + 4f8ffd1
commit 7a95457
Show file tree

Hide file tree

Showing 13 changed files with 24 additions and 119 deletions.
diff --git a/README.md b/README.md
@@ -152,5 +152,3 @@ heroku run rails db:seed
 * Sai Nithin
 * Vinayaka Hegde
 * [Legacy Code](https://github.com/tamu-edu-students/csce606-ELRC-Synergistic-Leadership-Theory)
-
-
diff --git a/rails_root/app/controllers/admins_controller.rb b/rails_root/app/controllers/admins_controller.rb
@@ -1,6 +1,4 @@
-# frozen_string_literal: true
-
-# admins_controller.rb
+#admins_controller.rb
 class AdminsController < ApplicationController
   def index
     @survey_responses = SurveyResponse.get_all_responses(page: params[:page], per_page: 20)

diff --git a/rails_root/app/helpers/admins_helper.rb b/rails_root/app/helpers/admins_helper.rb
@@ -1,5 +1,2 @@
-# frozen_string_literal: true
-
-# helper method for the admin controller
 module AdminsHelper
 end
diff --git a/rails_root/app/models/survey_response.rb b/rails_root/app/models/survey_response.rb
@@ -16,10 +16,10 @@ class SurveyResponse < ApplicationRecord
   belongs_to :profile,
              class_name: 'SurveyProfile'
 
-  has_many :invitations,
-           foreign_key: :parent_response_id,
-           class_name: 'Invitation',
-           dependent: :destroy
+  has_many :invitations, 
+            foreign_key: :parent_response_id, 
+            class_name: 'Invitation',
+            dependent: :destroy
 
   def self.create_from_params(user_id, params)
     # FIXME: When we look up things and fail, we should use more descriptive exceptions instead of ActiveRecord::RecordNotFound

diff --git a/rails_root/app/views/admins/index.html.erb b/rails_root/app/views/admins/index.html.erb
@@ -8,26 +8,22 @@
     <div class="table-responsive">
       <table class = "table">
         <thead>
-          <tr>
-            <th>ID</th>
-            <th>Profile</th>
-            <th>Share Code</th>
-            <th>Created At</th>
-            <th>View Survey</th> 
-          </tr>
+            <tr>
+                <th>ID</th>
+                <th>Profile</th>
+                <th>Share Code</th>
+                <th>Created At</th>
+            </tr>
         </thead>
         <% if @survey_responses.present? %>
           <tbody>
             <% @survey_responses.each do |response| %>
-              <tr>
-                <td><%= response.id %></td>
-                <td><%= response.profile.user_id %></td>
-                <td><%= response.share_code %></td>
-                <td><%= response.created_at.strftime('%Y-%m-%d %H:%M:%S') %></td>
-                <td>
-                <%= link_to 'View Survey', survey_response_path(response) %>
-                </td>
-              </tr>
+                <tr>
+                    <td><%= response.id %></td>
+                    <td><%= response.profile.user_id %></td>
+                    <td><%= response.share_code %></td>
+                    <td><%= response.created_at.strftime('%Y-%m-%d %H:%M:%S') %></td>
+                </tr>
             <% end %>
           </tbody>
         <% end %>

diff --git a/rails_root/app/views/layouts/_header.html.erb b/rails_root/app/views/layouts/_header.html.erb
@@ -23,10 +23,6 @@
       <li class="nav-item">
         <%= link_to "About", about_path, class: "nav-link #{'active' if current_page?(about_path)}" %>
       </li>
-      <li class="nav-item">
-        <%= link_to "Admin Dashboard", admin_dashboard_path, class: "nav-link" %> <!-- Admin Dashboard Link -->
-      </li>
-
       <li class="nav-item">
         <%# <%= link_to "Survey Responses", survey_responses_path, class: "nav-link #{'active' if current_page?(survey_responses_path)}" %>
       </li>

diff --git a/rails_root/config/routes.rb b/rails_root/config/routes.rb
@@ -8,9 +8,7 @@
 
   get 'home/index'
   get 'about', to: 'about#index'
-  get '/admin', to: 'admins#index', as: 'admin_dashboard'
-  #get 'survey_responses/:id', to: 'survey_responses#show', as: 'survey_response'
-
+  get '/admin', to: 'admins#index'
 
   # get 'survey', to: 'survey_responses#new', as: 'survey'
   # get 'survey/page/:page', to: 'survey_responses#survey', as: 'survey_page'

diff --git a/rails_root/features/admin_management.feature b/rails_root/features/admin_management.feature
diff --git a/rails_root/features/step_definitions/admin_management_steps.rb b/rails_root/features/step_definitions/admin_management_steps.rb
diff --git a/rails_root/spec/helpers/admins_helper_spec.rb b/rails_root/spec/helpers/admins_helper_spec.rb
@@ -1,5 +1,3 @@
-# frozen_string_literal: true
-
 require 'rails_helper'
 
 # Specs in this file have access to a helper object that includes

diff --git a/rails_root/spec/requests/admin_dashboard_spec.rb b/rails_root/spec/requests/admin_dashboard_spec.rb
diff --git a/rails_root/spec/requests/admins_spec.rb b/rails_root/spec/requests/admins_spec.rb
@@ -1,12 +1,11 @@
-# frozen_string_literal: true
-
 require 'rails_helper'
 
-RSpec.describe 'Admins', type: :request do
-  describe 'GET /index' do
-    it 'returns http success' do
-      get '/admins/index'
+RSpec.describe "Admins", type: :request do
+  describe "GET /index" do
+    it "returns http success" do
+      get "/admins/index"
       expect(response).to have_http_status(:success)
     end
   end
+
 end
diff --git a/rails_root/spec/views/admins/index.html.erb_spec.rb b/rails_root/spec/views/admins/index.html.erb_spec.rb
@@ -1,7 +1,5 @@
-# frozen_string_literal: true
-
 require 'rails_helper'
 
-RSpec.describe 'admins/index.html.erb', type: :view do
+RSpec.describe "admins/index.html.erb", type: :view do
   pending "add some examples to (or delete) #{__FILE__}"
 end