maverick/docbrown
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Restore some context in comment (#11587)

Browse source 
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.
This commit is contained in:
Jacob Herrington 2020-11-24 09:08:41 -06:00 • committed by GitHub
parent c29bff5955
commit e2f33a37b7
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
app/views/users/_sidebar.html.erb
View file

@ -7,6 +7,7 @@
</header>
<div>
<%# 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| %>
<a href="/<%= organization.slug %>" class="flex items-center crayons-link crayons-link--contentful">
<span class="crayons-logo crayons-logo--l mr-2 shrink-0">