WIP: finished up building the form

It's better than it was. Requesting clients will need to pass some kind of callback_url, and be ready to process it when it returns

Author: jonodrew

app/controllers/admin/member_search_controller.rb

@@ -3,11 +3,17 @@ def index
     @params = params[:member_search] || {}
     @name = @params[:name]
     @members = @name.blank? ? Member.none : Member.find_members(@name).select(:id, :name, :surname, :pronouns)
-    @callback_url = @params[:callback] || root_path
+    @callback_url = @params[:callback] || results_admin_member_search_index_path
     if (@members.size == 1) && @callback_url.present?
       return redirect_to @callback_url, member_id: @members.first.id
     end
 
     render 'index', locals: { members: @members, callback: @callback_url }
   end
+
+  def results
+    params.require(:member_pick).permit(members: [])
+    @members = params[:member_pick].present? ? Member.find(params[:member_pick][:members]) : Member.none
+    render 'show', locals: { members: @members }
+  end
 end

app/views/admin/member_search/_search_form.html.haml

@@ -2,4 +2,4 @@
   .row 
     .col-12.col-md-6
       = f.input :name, label: 'Member Name', input_html: { placeholder: 'Enter member name' }
-      = f.button :submit, 'Search', class: 'btn btn-primary mt-3'
+      = f.button :submit, 'Search', class: 'btn btn-primary mt-3, name: nil

app/views/admin/member_search/index.html.haml

@@ -11,10 +11,13 @@
     .col.col-md-10.col-lg-8
       - if @members.present?
         %h2 Select Member 
-        
-        = simple_form_for :member_pick, url: @callback_url, method: :get, html: {:multipart => true, novalidate: true } do |f|
+        = simple_form_for :member_pick, url: results_admin_member_search_index_path, method: :get do |f|
           .row 
             .col-12.col-md-6
-            = f.input :name, collection: @members, label: 'Member Name', input_html: { multiple: true }, required: true
-            = f.button :submit, 'Take me back', class: 'btn btn-primary mt-3'
+            = f.label :members, 'Member Name'
+            - @members.each do |member|
+              .form-check
+                = f.check_box :members, { multiple: true, checked: member.id }, member.id, nil
+                = f.label "member_#{member.id}", member.name_and_surname, class: 'form-check-label'
+            = f.button :submit, 'Take me back', class: 'btn btn-primary mt-3', name: nil
 

app/views/admin/member_search/show.html.haml

@@ -0,0 +1,12 @@
+.container.py-4.py-lg-5
+  .row.mb-4
+    .col
+      %h1 Results 
+
+    .row
+      .col.col-md-10.col-lg-8
+        %h2 Search Results
+        %ul.list-group
+          - @members.each do |member|
+            %li.list-group-item
+              = link_to member.name_and_surname, admin_member_path(member)

config/routes.rb

@@ -141,7 +141,12 @@
 
     resources :testimonials, only: %i[index]
 
-    resources :member_search, only: %i[index],  :path => 'member-search'
+    resources :member_search, :path => 'member-search', only: [:index, :results] do
+      collection do
+        get 'index'
+        get 'results'
+      end
+    end
   end
 
   get   '/login', to: 'auth_services#new'