feat(admin): batch convert controllers to params.expect

Converted controllers:
- workshops_controller: array params, host_id helper method
- events_controller: multiple nested arrays for sponsor tiers
- sponsors_controller: nested attributes (address, contacts)
- announcements_controller: group_ids array
- chapters_controller: basic fields
- groups_controller: basic fields
- meetings_controller: multiple fields
- bans_controller: ban params
- member_notes_controller: note params

All controllers now use params.expect syntax instead of require().permit().
All existing tests pass with no regressions.

Author: mroderick

app/controllers/admin/announcements_controller.rb

@@ -36,6 +36,6 @@ def set_announcement
   end
 
   def announcement_params
-    params.require(:announcement).permit(:all_groups, :message, :expires_at, group_ids: [])
+    params.expect(announcement: [:all_groups, :message, :expires_at, { group_ids: [] }])
   end
 end

app/controllers/admin/bans_controller.rb

@@ -20,7 +20,7 @@ def create
   private
 
   def ban_params
-    params.require(:ban).permit(:note, :reason, :permanent, :expires_at, :explanation)
+    params.expect(ban: [:note, :reason, :permanent, :expires_at, :explanation])
   end
 
   def set_member

app/controllers/admin/chapters_controller.rb

@@ -58,7 +58,7 @@ def members
   private
 
   def chapter_params
-    params.require(:chapter).permit(:name, :email, :city, :time_zone, :description, :image)
+    params.expect(chapter: [:name, :email, :city, :time_zone, :description, :image])
   end
 
   def set_chapter

app/controllers/admin/events_controller.rb

@@ -72,14 +72,17 @@ def set_event
   end
 
   def event_params
-    params.require(:event).permit(
+    params.expect(event: [
       :virtual, :name, :slug, :date_and_time, :local_date, :local_time, :local_end_time, :description,
       :info, :schedule, :venue_id, :external_url, :coach_spaces, :student_spaces, :email, :announce_only,
       :tito_url, :invitable, :time_zone, :student_questionnaire, :confirmation_required, :surveys_required,
       :audience, :coach_questionnaire, :show_faq, :display_coaches, :display_students,
-      bronze_sponsor_ids: [], silver_sponsor_ids: [], gold_sponsor_ids: [], sponsor_ids: [],
-      chapter_ids: []
-    )
+      { bronze_sponsor_ids: [] },
+      { silver_sponsor_ids: [] },
+      { gold_sponsor_ids: [] },
+      { sponsor_ids: [] },
+      { chapter_ids: [] }
+    ])
   end
 
   def organiser_ids

app/controllers/admin/groups_controller.rb

@@ -27,6 +27,6 @@ def show
   private
 
   def group_params
-    params.require(:group).permit(:name, :description, :chapter_id)
+    params.expect(group: [:name, :description, :chapter_id])
   end
 end

app/controllers/admin/meetings_controller.rb

@@ -62,10 +62,10 @@ def slug
   end
 
   def meeting_params
-    params.require(:meeting).permit(
+    params.expect(meeting: [
       :name, :description, :slug, :date_and_time, :local_date, :local_time, :local_end_time,
       :invitable, :spaces, :venue_id, :sponsor_id, :chapters
-    )
+    ])
   end
 
   def organiser_ids

app/controllers/admin/member_notes_controller.rb

@@ -9,6 +9,6 @@ def create
   end
 
   def member_note_params
-    params.require(:member_note).permit(:note, :member_id)
+    params.expect(member_note: [:note, :member_id])
   end
 end

app/controllers/admin/sponsors_controller.rb

@@ -63,12 +63,12 @@ def update
   private
 
   def sponsor_params
-    params.require(:sponsor).permit(:name, :avatar, :website, :seats, :accessibility_info,
-                                    :number_of_coaches, :level, :description,
-                                    address_attributes: %i[id flat street postal_code city
-                                                           latitude longitude directions],
-                                    contacts_attributes: %i[id name surname email mailing_list_consent
-                                                            _destroy])
+    params.require(:sponsor).permit(
+      :name, :avatar, :website, :seats, :accessibility_info,
+      :number_of_coaches, :level, :description,
+      address_attributes: [:id, :flat, :street, :postal_code, :city, :latitude, :longitude, :directions],
+      contacts_attributes: [:id, :name, :surname, :email, :mailing_list_consent, :_destroy]
+    )
   end
 
   def set_sponsor

app/controllers/admin/workshops_controller.rb

@@ -117,10 +117,13 @@ def changes
   private
 
   def workshop_params
-    params.require(:workshop).permit(:local_date, :local_time, :local_end_time, :chapter_id,
-                                     :invitable, :seats, :virtual, :slack_channel, :slack_channel_link,
-                                     :rsvp_open_local_date, :rsvp_open_local_time, :description,
-                                     :coach_spaces, :student_spaces, sponsor_ids: [])
+    params.expect(workshop: [
+      :local_date, :local_time, :local_end_time, :chapter_id,
+      :invitable, :seats, :virtual, :slack_channel, :slack_channel_link,
+      :rsvp_open_local_date, :rsvp_open_local_time, :description,
+      :coach_spaces, :student_spaces,
+      { sponsor_ids: [] }
+    ])
   end
 
   def chapter_id
@@ -157,7 +160,7 @@ def set_organisers(organiser_ids)
   end
 
   def host_id
-    params.require(:workshop).permit(:host)[:host]
+    params.expect(workshop: [:host])[:host]
   end
 
   def organiser_ids