Skip to content

Commit 653954a

Browse files
committed
Rename paginated overall leaderboard XP to just overall leaderboard XP
1 parent cc9ddac commit 653954a

File tree

4 files changed

+10
-10
lines changed

4 files changed

+10
-10
lines changed

src/commons/sagas/LeaderboardSaga.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,18 +9,18 @@ import {
99
getAllContests,
1010
getContestPopularVoteLeaderboard,
1111
getContestScoreLeaderboard,
12-
getPaginatedTotalXp
12+
getOverallLeaderboardXP
1313
} from './RequestsSaga';
1414

1515
const LeaderboardSaga = combineSagaHandlers({
16-
[LeaderboardActions.getPaginatedLeaderboardXp.type]: function* (action) {
16+
[LeaderboardActions.getOverallLeaderboardXP.type]: function* (action) {
1717
const tokens: Tokens = yield selectTokens();
1818
const { page, pageSize } = action.payload;
1919

20-
const paginatedUsersXp = yield call(getPaginatedTotalXp, page, pageSize, tokens);
20+
const paginatedUsersXp = yield call(getOverallLeaderboardXP, page, pageSize, tokens);
2121

2222
if (paginatedUsersXp) {
23-
yield put(actions.savePaginatedLeaderboardXp(paginatedUsersXp));
23+
yield put(actions.saveOverallLeaderboardXP(paginatedUsersXp));
2424
}
2525
},
2626

src/commons/sagas/RequestsSaga.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -491,16 +491,16 @@ export const getAllOverallLeaderboardXP = async (
491491
};
492492

493493
/**
494-
* GET /courses/{courseId}/get_paginated_display
494+
* GET /courses/{courseId}/leaderboards/xp
495495
*/
496-
export const getPaginatedTotalXp = async (
496+
export const getOverallLeaderboardXP = async (
497497
page: number,
498498
pageSize: number,
499499
tokens: Tokens
500500
): Promise<{ rows: LeaderboardRow[]; userCount: number } | null> => {
501501
const offset = (page - 1) * pageSize;
502502
const params = new URLSearchParams({ offset: `${offset}`, page_size: `${pageSize}` });
503-
const resp = await request(`${courseId()}/get_paginated_display?${params.toString()}`, 'GET', {
503+
const resp = await request(`${courseId()}/leaderboards/xp?${params.toString()}`, 'GET', {
504504
...tokens
505505
});
506506

src/features/leaderboard/LeaderboardActions.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,8 @@ import {
77
} from './LeaderboardTypes';
88

99
const LeaderboardActions = createActions('leaderboard', {
10-
getPaginatedLeaderboardXp: (page: number, pageSize: number) => ({ page, pageSize }),
11-
savePaginatedLeaderboardXp: (payload: { rows: LeaderboardRow[]; userCount: number }) => payload,
10+
getOverallLeaderboardXP: (page: number, pageSize: number) => ({ page, pageSize }),
11+
saveOverallLeaderboardXP: (payload: { rows: LeaderboardRow[]; userCount: number }) => payload,
1212
getAllContestScores: (assessmentId: number, visibleEntries: number) => ({
1313
assessmentId,
1414
visibleEntries

src/features/leaderboard/LeaderboardReducer.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ export const LeaderboardReducer: Reducer<LeaderboardState, SourceActionType> = c
99
defaultLeaderboard,
1010
builder => {
1111
builder
12-
.addCase(LeaderboardActions.savePaginatedLeaderboardXp, (state, action) => {
12+
.addCase(LeaderboardActions.saveOverallLeaderboardXP, (state, action) => {
1313
state.paginatedUserXp = {
1414
rows: action.payload.rows || [],
1515
userCount: action.payload.userCount || 0

0 commit comments

Comments
 (0)