add coaches to events

Author: simaofreitas

app/controllers/admin/groups_controller.rb

@@ -8,14 +8,22 @@ def index
   end
 
   def generate
+    # This needs to be wrapped in a transaction
     @attendees = @event.applications.application_selected.confirmed
     @attendees.each_slice(6).with_index do |group, index|
-      event_group = EventGroup.create(event: @event, name: "Group #{index}")
+      event_group = EventGroup.create(event: @event, name: "Group #{index + 1}")
       group.each do |application|
         event_group.applications << application
       end
     end
 
+    @coaches = @event.coach_applications.approved.to_a
+    @event.event_groups.each do |event_group|
+      # Check if we are assinging the last 2 coaches
+      coach_group = @coaches.pop(2)
+      event_group.coach_applications << coach_group unless coach_group.empty?
+    end
+
     @event_groups = @event.event_groups
 
     redirect_to admin_event_groups_path(@event), notice: "Groups successfully generated"

app/models/coach_application.rb

@@ -5,6 +5,7 @@ class CoachApplication < ApplicationRecord
   accepts_nested_attributes_for :event
 
   scope :to_contact, -> { where(contacted_at: nil, state: :approved) }
+  scope :approved, ->{ where(state: :approved) }
 
   enum state: { pending: 0, approved: 1, rejected: 2, cancelled: 3 }
 

app/models/event_group.rb

@@ -1,5 +1,5 @@
 class EventGroup < ApplicationRecord
   belongs_to :event
-  has_and_belongs_to_many :coaches
+  has_and_belongs_to_many :coach_applications, join_table: "event_groups_coach_applications"
   has_and_belongs_to_many :applications, join_table: "event_groups_applications"
 end

app/views/admin/groups/index.html.erb

@@ -1,26 +1,26 @@
 <% if @event.has_groups? %>
-has Groups
 
-<% @event_groups.each do |event_group| %>
-<h2><%= event_group.name %></h2>
-<table>
-  <thead>
-    <tr>
-      <th>Attendees</th>
-      <th>Coaches</th>
-    </tr>
-  </thead>
-  <tbody>
-    <tr>
-      <td>
-        <%= event_group.applications.map(&:name).join(", ") %>
-      </td>
-      <td>
-      </td>
-    </tr>
-  </tbody>
-</table>
-<%end %>
+  <% @event_groups.each do |event_group| %>
+    <h2><%= event_group.name %></h2>
+    <table>
+      <thead>
+        <tr>
+          <th>Attendees</th>
+          <th>Coaches</th>
+        </tr>
+      </thead>
+      <tbody>
+        <tr>
+          <td>
+            <%= event_group.applications.map(&:name).join(", ") %>
+          </td>
+          <td>
+
+          </td>
+        </tr>
+      </tbody>
+    </table>
+  <%end %>
 
 <% else %>
   <%= button_to "Generate groups", generate_admin_event_groups_path(@event), method: :post %>

db/migrate/20200429185626_change_event_groups_coaches_to_event_groups_coach_applications.rb

@@ -0,0 +1,5 @@
+class ChangeEventGroupsCoachesToEventGroupsCoachApplications < ActiveRecord::Migration[5.2]
+  def change
+    rename_table :event_groups_coaches, :event_groups_coach_applications
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2020_04_23_182618) do
+ActiveRecord::Schema.define(version: 2020_04_29_185626) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -88,13 +88,13 @@
     t.index ["event_group_id"], name: "index_event_groups_applications_on_event_group_id"
   end
 
-  create_table "event_groups_coaches", force: :cascade do |t|
+  create_table "event_groups_coach_applications", force: :cascade do |t|
     t.bigint "coach_application_id"
     t.bigint "event_group_id"
     t.datetime "created_at", null: false
     t.datetime "updated_at", null: false
-    t.index ["coach_application_id"], name: "index_event_groups_coaches_on_coach_application_id"
-    t.index ["event_group_id"], name: "index_event_groups_coaches_on_event_group_id"
+    t.index ["coach_application_id"], name: "index_event_groups_coach_applications_on_coach_application_id"
+    t.index ["event_group_id"], name: "index_event_groups_coach_applications_on_event_group_id"
   end
 
   create_table "events", id: :serial, force: :cascade do |t|
@@ -152,6 +152,6 @@
   add_foreign_key "event_groups", "events"
   add_foreign_key "event_groups_applications", "applications"
   add_foreign_key "event_groups_applications", "event_groups"
-  add_foreign_key "event_groups_coaches", "coach_applications"
-  add_foreign_key "event_groups_coaches", "event_groups"
+  add_foreign_key "event_groups_coach_applications", "coach_applications"
+  add_foreign_key "event_groups_coach_applications", "event_groups"
 end