rubyvideo icon indicating copy to clipboard operation
rubyvideo copied to clipboard
verified publisher icon adrienpoly

Fix Talk#Show speakers n+1

Open justinallenmarsh opened this issue 1 year ago • 3 comments

There is an n+1 query triggered by rendering the speakers of each talk because it's not eager loaded in the controller when rendering Speakers#show

Before After
Before After

justinallenmarsh avatar Oct 03 '24 15:10 justinallenmarsh

Thanks but I think the PR is missing the commit

adrienpoly avatar Oct 04 '24 05:10 adrienpoly

@adrienpoly I added the missing commit 😇

justinallenmarsh avatar Oct 04 '24 10:10 justinallenmarsh

@adrienpoly I think this one is also ready to merge with approval.

justinallenmarsh avatar Oct 06 '24 10:10 justinallenmarsh