The spell check error has been fixed on #12581.

@alecslupu do you have any way of validating this one? I'm trying to see it through the bullet queries. I don't see much difference in the number of queries.

Not only that, but I'm also trying with https://github.com/brunofacca/active-record-query-trace. I don't see much difference between this PR and develop.

Do you have any script or screenshot to check out the performance improvement?

Do you have any script or screenshot to check out the performance improvement?

@andreslucena
I have managed to use factory_bot for it. If you apply the below snippet, you should be able to view some improvements. There is a N+1 query caused by up_votes.any? { |vote| vote.author == user }.

When trying to render the comments list using the develop branch you should see a lot of SELECT * FROM decidim_users ... , whereas when applying this branch, you should not see those queries.

require "faker"
require "factory_bot"
require "decidim/core/test/factories"
require "decidim/participatory_processes/test/factories"
require "decidim/comments/test/factories"

Decidim::Comments::Comment.all.each do |comment|
  FactoryBot.create_list(:comment_vote, 50, :up_vote, comment: comment)
  FactoryBot.create_list(:comment_vote, 50, :down_vote, comment: comment)
end