Lift upper limit on list leaderboard records up to 1,000 per page.

heroiclabs · Mar 14, 2024 · aacd69c · aacd69c
1 parent 676ebb7
commit aacd69c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,7 +5,8 @@ The format is based on [keep a changelog](http://keepachangelog.com) and this pr
 
 ## [Unreleased]
 ### Changes
-- Reduce number of memory allocations in leaderboard cache
+- Reduce number of memory allocations in leaderboard cache.
+- List leaderboard records can now return up to 1,000 records per request.
 
 ### Fixed
 - Prevent players from requesting duplicate joins to the same party.

diff --git a/server/api_leaderboard.go b/server/api_leaderboard.go
@@ -112,8 +112,8 @@ func (s *ApiServer) ListLeaderboardRecords(ctx context.Context, in *api.ListLead
 
 	var limit *wrapperspb.Int32Value
 	if in.GetLimit() != nil {
-		if in.GetLimit().Value < 1 || in.GetLimit().Value > 100 {
-			return nil, status.Error(codes.InvalidArgument, "Invalid limit - limit must be between 1 and 100.")
+		if in.GetLimit().Value < 1 || in.GetLimit().Value > 1000 {
+			return nil, status.Error(codes.InvalidArgument, "Invalid limit - limit must be between 1 and 1000.")
 		}
 		limit = in.GetLimit()
 	} else if len(in.GetOwnerIds()) == 0 || in.GetCursor() != "" {