From e2f33a37b70719fba5f02b1af3969ea304103237 Mon Sep 17 00:00:00 2001 From: Jacob Herrington Date: Tue, 24 Nov 2020 09:08:41 -0600 Subject: [PATCH] Restore some context in comment (#11587) rhymes mentioned that this context might be useful regarding this comment. I removed some of this info in https://github.com/forem/forem/pull/11480, so this commit adds that context along with a link that rhymes shared with me. --- app/views/users/_sidebar.html.erb | 1 + 1 file changed, 1 insertion(+) diff --git a/app/views/users/_sidebar.html.erb b/app/views/users/_sidebar.html.erb index b2cb6e58e..1a886afb4 100644 --- a/app/views/users/_sidebar.html.erb +++ b/app/views/users/_sidebar.html.erb @@ -7,6 +7,7 @@
<%# it's unlikely that a user will belong to enough organizations to cause a performance issue here %> + <%# More info regarding the `.present?/.each` pattern: https://www.speedshop.co/2019/01/10/three-activerecord-mistakes.html %> <% @user.organizations.each do |organization| %>