refactor(events): remove unnecessary text translations preload and update includes in event policy

rsmithlal · rsmithlal · commit 657e91f0bee0 · 2025-10-27T20:30:38.000-02:30
diff --git a/app/controllers/better_together/events_controller.rb b/app/controllers/better_together/events_controller.rb
@@ -325,7 +325,6 @@ def preload_event_associations! # rubocop:todo Metrics/CyclomaticComplexity, Met
 
       # Preload translations for the event itself
       @event.string_translations.load
-      @event.text_translations.load
 
       # Preload cover image attachment to avoid attachment queries
       @event.cover_image_attachment&.blob&.load if @event.cover_image.attached?
diff --git a/app/policies/better_together/event_policy.rb b/app/policies/better_together/event_policy.rb
@@ -57,7 +57,7 @@ def event_host_member?
     class Scope < ApplicationPolicy::Scope
       def resolve
         scope.with_attached_cover_image
-             .includes(:string_translations, :text_translations, :location, :event_hosts, categorizations: {
+             .includes(:string_translations, :location, :event_hosts, categorizations: {
                          category: %i[
                            string_translations cover_image_attachment cover_image_blob
                          ]

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ def event_host_member?`
`57`	`57`	`class Scope < ApplicationPolicy::Scope`
`58`	`58`	`def resolve`
`59`	`59`	`scope.with_attached_cover_image`
`60`		`- .includes(:string_translations, :text_translations, :location, :event_hosts, categorizations: {`
	`60`	`+ .includes(:string_translations, :location, :event_hosts, categorizations: {`
`61`	`61`	`category: %i[`
`62`	`62`	`string_translations cover_image_attachment cover_image_blob`
`63`	`63`	`]`