Remove action and reducer to get all users for overall leaderboards

Author: sayomaki

src/commons/sagas/LeaderboardSaga.ts

@@ -7,23 +7,12 @@ import { actions } from '../utils/ActionsHelper';
 import { selectTokens } from './BackendSaga';
 import {
   getAllContests,
-  getAllTotalXp,
   getContestPopularVoteLeaderboard,
   getContestScoreLeaderboard,
   getPaginatedTotalXp
 } from './RequestsSaga';
 
 const LeaderboardSaga = combineSagaHandlers({
-  [LeaderboardActions.getAllUsersXp.type]: function* () {
-    const tokens: Tokens = yield selectTokens();
-
-    const usersXp = yield call(getAllTotalXp, tokens);
-
-    if (usersXp) {
-      yield put(actions.saveAllUsersXp(usersXp));
-    }
-  },
-
   [LeaderboardActions.getPaginatedLeaderboardXp.type]: function* (action) {
     const tokens: Tokens = yield selectTokens();
     const { page, pageSize } = action.payload;

src/commons/sagas/RequestsSaga.ts

@@ -463,10 +463,12 @@ export const getTotalXp = async (tokens: Tokens, courseRegId?: number): Promise<
 };
 
 /**
- * GET /courses/{courseId}/all_user_xp
+ * GET /courses/{courseId}/leaderboards/xp_all
  */
-export const getAllTotalXp = async (tokens: Tokens): Promise<number | null> => {
-  const resp = await request(`${courseId()}/all_users_xp`, 'GET', {
+export const getAllOverallLeaderboardXP = async (
+  tokens: Tokens
+): Promise<LeaderboardRow[] | null> => {
+  const resp = await request(`${courseId()}/leaderboards/xp_all`, 'GET', {
     ...tokens
   });
 

src/features/leaderboard/LeaderboardActions.ts

@@ -7,8 +7,6 @@ import {
 } from './LeaderboardTypes';
 
 const LeaderboardActions = createActions('leaderboard', {
-  getAllUsersXp: 0,
-  saveAllUsersXp: (userXp: LeaderboardRow[]) => userXp,
   getPaginatedLeaderboardXp: (page: number, pageSize: number) => ({ page, pageSize }),
   savePaginatedLeaderboardXp: (payload: { rows: LeaderboardRow[]; userCount: number }) => payload,
   getAllContestScores: (assessmentId: number, visibleEntries: number) => ({

src/features/leaderboard/LeaderboardReducer.ts

@@ -9,9 +9,6 @@ export const LeaderboardReducer: Reducer<LeaderboardState, SourceActionType> = c
   defaultLeaderboard,
   builder => {
     builder
-      .addCase(LeaderboardActions.saveAllUsersXp, (state, action) => {
-        state.userXp = action.payload;
-      })
       .addCase(LeaderboardActions.savePaginatedLeaderboardXp, (state, action) => {
         state.paginatedUserXp = {
           rows: action.payload.rows || [],