diff --git a/app/controllers/admin/feedback_messages_controller.rb b/app/controllers/admin/feedback_messages_controller.rb index 6b88ff243..53bf478f9 100644 --- a/app/controllers/admin/feedback_messages_controller.rb +++ b/app/controllers/admin/feedback_messages_controller.rb @@ -121,10 +121,15 @@ module Admin end def reconcile_ransack_params - if (params[:q] && params.dig(:q, :status_eq).blank?) && params[:status].present? + params[:q] ||= {} + if params.dig(:q, :status_eq).blank? && params[:status].present? params[:q][:status_eq] = params[:status] end - params[:status] = params[:q][:status_eq] if params[:q].present? # ransack + if params.dig(:q, :category_eq).blank? && params[:category].present? + params[:q][:category_eq] = params[:category] + end + params[:status] = params[:q][:status_eq] if params[:q].present? + params[:category] = params[:q][:category_eq] if params[:q].present? end end end diff --git a/app/views/admin/feedback_messages/_feedback_message.html.erb b/app/views/admin/feedback_messages/_feedback_message.html.erb index 3f924689b..1fdaac928 100644 --- a/app/views/admin/feedback_messages/_feedback_message.html.erb +++ b/app/views/admin/feedback_messages/_feedback_message.html.erb @@ -4,15 +4,17 @@ <% feedback_message_emails = @email_messages.select { |email| email.feedback_message_id == feedback_message.id } %> <% end %> -
+
Report #<%= feedback_message.id %> - Submitted: <%= time_ago_in_words feedback_message.created_at %> ago
+ <%= feedback_message.category.titleize %> ✉️ Emails Sent: <%= feedback_message_emails.size %> | <%= link_to "Link to Report #{feedback_message.id}", admin_report_path(feedback_message.id) %> +
@@ -60,11 +62,6 @@ <% end %> -
-

- <%= feedback_message.category.titleize %> -

-
<% if feedback_message.offender %>

diff --git a/app/views/admin/feedback_messages/index.html.erb b/app/views/admin/feedback_messages/index.html.erb index 7be79e5b3..dfdea2125 100644 --- a/app/views/admin/feedback_messages/index.html.erb +++ b/app/views/admin/feedback_messages/index.html.erb @@ -11,6 +11,9 @@ <%= f.label :reporter_username_cont, "Reporter", class: "crayons-field__label m-1 screen-reader-only" %> <%= f.search_field :reporter_username_cont, placeholder: "Reporter", class: "crayons-textfield w-auto m-1" %> <%= f.select(:status_eq, options_for_select(%w[Open Invalid Resolved], params[:status] || "Open"), { include_blank: true }, class: "crayons-select w-auto m-1") %> + <%= f.label :category_eq, "Category", class: "crayons-field__label m-1 screen-reader-only" %> +<%= f.select :category_eq, options_for_select(FeedbackMessage::CATEGORIES, params[:category]), { include_blank: true }, class: "crayons-select w-auto m-1" %> + <%= f.submit "Search", class: "crayons-btn crayons-btn--secondary m-1" %> <% end %>