ylqjgm
/
canvas-lms
mirror of https://github.com/instructure/canvas-lms.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

actually preload the association we use 

Change-Id: I95c9e0ee2ca4191f5ca87f8f2fcf9df15a84fc59
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/231010
Reviewed-by: James Williams <jamesw@instructure.com>
Reviewed-by: Simon Williams <simon@instructure.com>
Reviewed-by: Rob Orton <rob@instructure.com>
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
QA-Review: Rob Orton <rob@instructure.com>
Product-Review: Rob Orton <rob@instructure.com>
This commit is contained in:
Cody Cutrer 2020-03-23 11:14:15 -06:00
parent 56da7cc52c
commit be33585096
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/models/delayed_notification.rb
View File

@ -79,7 +79,7 @@ class DelayedNotification < ActiveRecord::Base
lookups.each do |klass, ids|
includes = []
includes = [ :notification_policies, { user: :pseudonyms } ] if klass == CommunicationChannel
includes = [ :pseudonyms, { communication_channel: :notification_policies } ] if klass == User
includes = [ :pseudonyms, { communication_channels: :notification_policies } ] if klass == User
ids.each_slice(100) do |slice|
yield klass.where(:id => slice).preload(includes).to_a