@@ -8,7 +8,10 @@ class FeedbackController < ApplicationController
     unless form_values.empty?
       FeedbackMailer.feedback(current_user, form_values).deliver_later
     end
-    # @todo - redirect somewhere to avoid back/refresh button issues
+    redirect_to '/feedback/thanks'
+  end
+
+  def thanks
   end
 
   protected
class FeedbackController < ApplicationController
  def index
    @skip_footer = true
  end

  def create
    form_values = self.feedback_params
    unless form_values.empty?
      FeedbackMailer.feedback(current_user, form_values).deliver_later
    end
    # @todo - redirect somewhere to avoid back/refresh button issues
  end

  protected

  def feedback_params
    params.permit(
      :email,
      :bug,
      :context,
      :expected,
      :actually,
      :comments,
      :satisfaction,
      :referral
    ).reject { |_key, val| val.blank? }
  end
end

When we settle on a support service, we may want to use an API rather than going through emails, which are harder to capture errors for.