fix: include invited coaches who never RSVPed in feedback dropdown

app/controllers/feedback_controller.rb

@@ -40,7 +40,8 @@ def feedback_params
   end
 
   def set_coaches(workshop)
-    @coaches = workshop.invitations.to_coaches.accepted_or_attended
+    @coaches = workshop.invitations.to_coaches
+      .where('attending IS NULL OR attending = ? OR attended = ?', true, true)
       .order(Arel.sql('attended DESC NULLS LAST'))
       .map(&:member)
   end

spec/features/member_feedback_spec.rb

@@ -57,6 +57,25 @@
       expect(page).to have_select('feedback_coach_id', with_options: [coach_not_yet_verified.full_name])
     end
 
+    scenario 'I can see coaches who were invited but never RSVPed (Issue #2633)' do
+      # This reproduces the real-world scenario where:
+      # 1. Coach is invited to workshop but never responds (attending = nil)
+      # 2. Coach shows up anyway
+      # 3. Organizer hasn't yet marked attendance (attended = nil)
+      # 4. Student receives feedback form and should still be able to select the coach
+      walk_in_coach = Fabricate(:coach)
+      Fabricate(:coach_workshop_invitation,
+        workshop: feedback_request.workshop,
+        member: walk_in_coach,
+        attending: nil,
+        attended: nil)
+
+      visit feedback_path(valid_token)
+
+      # Coach should appear in the list even though they never RSVPed
+      expect(page).to have_select('feedback_coach_id', with_options: [walk_in_coach.full_name])
+    end
+
     scenario 'verified coaches appear before unverified coaches in the list' do
       # Create two coaches: one verified (attended=true), one not yet (attended=nil)
       verified_coach = Fabricate(:coach, name: 'Alice', surname: 'Verified')