Skip to content

Commit afc9907

Browse files
authored
Leaderboard answers only from current week (#34)
* Update weekly leaderboard to only include answers from puzzles form that week * Rubocop fixes * Add joins
1 parent 12e3792 commit afc9907

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

app/jobs/weekly_leaderboard_job.rb

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,8 @@ def perform
77
end_time = Time.now.utc.end_of_week.end_of_day # Sunday at 23:59 UTC
88

99
leaderboard_data = Answer
10-
.where(created_at: start_time..end_time, is_correct: true)
10+
.joins(:puzzle)
11+
.where(created_at: start_time..end_time, is_correct: true, puzzles: { sent_at: start_time..end_time })
1112
.group(:server_id, :user_id)
1213
.order(Arel.sql("COUNT(*) DESC"))
1314
.count

0 commit comments

Comments
 (0)