Address Some Bullet Warnings / Optimise Mean Request Times by aaronskiba · Pull Request #3440 · DMPRoadmap/roadmap

added 4 commits

July 23, 2024 10:26
app/views/org_admin/plans/index.html.erb:
- .length loads the records into memory. Because we are eager-loading other tables (see code changes in app/controllers/org_admin/plans_controller.rb), this was triggering a "AVOID eager loading detected" Bullet warning. Because we are in fact making use of these related tables further in the code, the warnings themselves appear to be false-positives. However, the change to this file manages to suppresses the Bullet warnings, and is still very fast.

@aaronskiba aaronskiba changed the title Address Some Bullet Warnings / Optimise Page Loads Address Some Bullet Warnings / Optimise Mean Request Times

Jul 23, 2024

benjaminfaure

@aaronskiba