Eliminate n+1 query for user.tagnames

[Tlazypanda]

Please describe the desired behavior.
Wherever we are calling the function user.tagnames, the users don't include/eager-load the tag model which leads to n+1 queries.
Example: - list.html.erb line 70 (user.tags function is called which in turn calls the user.tagnames function)
Triggered from user.rb line line 161 method- tagnames
Can check this on /people route

Additional context (optional)
Feel free to include any applicable: code, screenshots, or gifs.

[sagarpreet-chadha]

Hi @Tlazypanda ,
So eager loading this query, we will make 1 huge query instead of n queries, is it?

[Tlazypanda]

Yeah 😅 but I am not sure if the code is right for this because it's not making much of a difference on my local setup so am not able to monitor. I guess after pushing we have to check if skylight can spot or any other way for this? 😅