show longer list

arkency · Oct 12, 2023 · e2d4bc2 · e2d4bc2
1 parent 3455c17
commit e2d4bc2
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/app/controllers/leaderboards_controller.rb b/app/controllers/leaderboards_controller.rb
@@ -1,6 +1,6 @@
 class LeaderboardsController < ApplicationController
   def show
     team = Team.find_by(slack_team_domain: params[:team_domain])
-    @leaderboard = Leaderboards.new(team).top_for_this_year
+    @leaderboard = Leaderboards.new(team, nil).top_for_this_year
   end
 end
diff --git a/app/models/leaderboards.rb b/app/models/leaderboards.rb
@@ -1,7 +1,8 @@
 class Leaderboards
-  def initialize(team)
+  def initialize(team, limit = 10)
     @team_time_zone = team.time_zone
     @team_scope = DailyStatistic.of_team(team.id)
+    @limit = limit
   end
 
   def top_for_this_week
@@ -22,7 +23,7 @@ def aggreated_data(scope)
     scope
       .group('user_name')
       .order('sum_points desc')
-      .limit(10)
+      .limit(@limit)
       .sum('points')
       .group_by(&:second)
       .map { |points, members| [points, members.map(&:first)] }.to_h
@@ -38,6 +39,7 @@ def method_missing(m, *args, &block)
     end
 
     private
+
     def format(hash)
       hash
         .map { |count, members| "#{count}: #{members.join(", ")}" }