Top Contributor missing the first user

saba
Fri, 01 Jul 2022 16:30:22 +0000
Bug

Thank you @meghna! Very cool and helpful component!

I added it to my Air Theme and the first user was missing from the list of top contributors. I think this should fix it:

github.com/discourse/discourse-top-contributors-sidebar

Fix: Do not skip top contributor in the list.

main ← sabajamalian:patch-1

merged 04:48PM - 01 Jul 22 UTC

sabajamalian

+1 -1

The JS slice function is inclusive of the starting point. The component currentl…y appears to be skipping the first user from the list of top contributors.

2 сообщения - 2 участника