Remove unused routes for feedback

We only need the post route

app/views/shared/_sul_header.html.erb

@@ -21,19 +21,17 @@
         <li>
           <%= link_to t('.my_account'), "https://mylibrary.stanford.edu/" %>
         </li>
-        <% unless current_page?(feedback_form_path) %>
-          <li>
-            <%= link_to "#feedback-form", role: 'button', data: {'bs-toggle':"collapse", 'bs-target':"#feedback-form"} do %>
-              Feedback
-            <% end %>
-          </li>
-        <% end %>
+        <li>
+          <%= link_to "#feedback-form", role: 'button', data: {'bs-toggle':"collapse", 'bs-target':"#feedback-form"} do %>
+            Feedback
+          <% end %>
+        </li>
       </ul>
     </nav>
   </div>
 </div>
 <div id="feedback-form" class="feedback-form-container collapse">
-    <%= render template: 'feedback_forms/new' unless current_page?(feedback_form_path) %>
+    <%= render 'feedback_forms/new' %>
 </div>
 <!-- Application header -->
 <div id="app-header" class="text-white bg-cardinal py-3 d-flex">

config/routes.rb

@@ -61,6 +61,5 @@
       post :comment, as: :comment
     end
   end
-  resource :feedback_form, path: 'feedback', only: %I[new, create]
-  get 'feedback' => 'feedback_forms#new'
+  resource :feedback_form, path: 'feedback', only: :create
 end

spec/requests/feedback_forms_spec.rb

@@ -2,13 +2,13 @@
 
 require 'rails_helper'
 
-RSpec.describe 'FeedbackForm', type: :feature do
+RSpec.describe 'The feedback form', type: :feature do
   let(:current_user?) { false }
   let(:user) { create(:sso_user) }
 
   context 'when not logged in' do
     it 'reCAPTCHA challenge is present' do
-      visit feedback_path
+      visit root_path
       expect(page).to have_css '.requests-captcha'
     end
   end
@@ -19,7 +19,7 @@
     end
 
     it 'reCAPTCHA challenge is NOT present' do
-      visit feedback_path
+      visit root_path
       expect(page).to have_no_css '.requests-captcha'
     end