chore: deleted unneeded comments

openedx · Jan 14, 2025 · 3428c00 · 3428c00
1 parent 31f3350
commit 3428c00
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 19 deletions.
diff --git a/src/components/PeopleManagement/EnterpriseCustomerUserDatatable.jsx b/src/components/PeopleManagement/EnterpriseCustomerUserDatatable.jsx
@@ -18,7 +18,7 @@ export const BaseSelectWithContext = ({ row, enterpriseGroupLearners }) => {
     checked,
     ...toggleRowSelectedProps
   } = row.getToggleRowSelectedProps();
-  const isAddedMember = enterpriseGroupLearners.find(learner => learner.enterpriseCustomerUserId === Number(row.id));
+  const isAddedMember = enterpriseGroupLearners.find(learner => learner.lmsUserId === Number(row.id));
   return (
     <div>
       <CheckboxControl
@@ -96,7 +96,7 @@ const EnterpriseCustomerUserDatatable = ({
       isSortable
       manualSortBy
       initialTableOptions={{
-        getRowId: row => row.enterpriseCustomerUser.name.toString(),
+        getRowId: row => row.enterpriseCustomerUser.userId.toString(),
       }}
       pageCount={enterpriseMembersTableData.pageCount}
       manualSelectColumn={

diff --git a/src/components/PeopleManagement/PeopleManagementTable.jsx b/src/components/PeopleManagement/PeopleManagementTable.jsx
@@ -37,7 +37,7 @@ const PeopleManagementTable = ({ enterpriseId }) => {
         pageSize: 10,
         pageIndex: 0,
         sortBy: [
-          { id: 'enterpriseCustomerUser.name', desc: true },
+          { id: 'name', desc: true },
         ],
         filters: [],
       }}

diff --git a/src/components/PeopleManagement/data/hooks/useEnterpriseMembersTableData.js b/src/components/PeopleManagement/data/hooks/useEnterpriseMembersTableData.js
@@ -35,21 +35,13 @@ const useEnterpriseMembersTableData = ({ enterpriseId }) => {
             options.user_query = value;
           }
         });
-        // if (args?.sortBy.length > 0) {
-        //   const sortByValue = args.sortBy[0].id;
-        //   options.sort_by = _.snakeCase(sortByValue);
-        // if (!args.sortBy[0].desc) {
-        //   options.is_reversed = !args.sortBy[0].desc;
-        // }
-
         if (args?.sortBy.length > 0) {
           const sortByValue = args.sortBy[0].id;
           options.sort_by = _.snakeCase(sortByValue);
           if (!args.sortBy[0].desc) {
             options.is_reversed = !args.sortBy[0].desc;
           }
         }
-        // }
         options.page = args.pageIndex + 1;
         const response = await LmsApiService.fetchEnterpriseCustomerMembers(enterpriseId, options);
         const data = camelCaseObject(response.data);

diff --git a/src/components/PeopleManagement/tests/AddMembersModal.test.jsx b/src/components/PeopleManagement/tests/AddMembersModal.test.jsx
@@ -70,31 +70,31 @@ const mockTabledata = {
   results: [
     {
       enterpriseCustomerUser: {
-        user_id: 1,
+        userId: 1,
         name: 'Test User 1',
         email: '[email protected]',
         joinedOrg: 'July 5, 2021',
       },
     },
     {
       enterpriseCustomerUser: {
-        user_id: 2,
+        userId: 2,
         name: 'Test User 2',
         email: '[email protected]',
         joinedOrg: 'July 2, 2022',
       },
     },
     {
       enterpriseCustomerUser: {
-        user_id: 3,
+        userId: 3,
         name: 'Test User 3',
         email: '[email protected]',
         joinedOrg: 'July 3, 2023',
       },
     },
     {
       enterpriseCustomerUser: {
-        user_id: 4,
+        userId: 4,
         name: 'Test User 4',
         email: '[email protected]',
         joinedOrg: 'July 4, 2024',

diff --git a/src/components/PeopleManagement/tests/CreateGroupModal.test.jsx b/src/components/PeopleManagement/tests/CreateGroupModal.test.jsx
@@ -67,31 +67,31 @@ const mockTabledata = {
   results: [
     {
       enterpriseCustomerUser: {
-        user_id: 1,
+        userId: 1,
         name: 'Test User 1',
         email: '[email protected]',
         joinedOrg: 'July 5, 2021',
       },
     },
     {
       enterpriseCustomerUser: {
-        user_id: 2,
+        userId: 2,
         name: 'Test User 2',
         email: '[email protected]',
         joinedOrg: 'July 2, 2022',
       },
     },
     {
       enterpriseCustomerUser: {
-        user_id: 3,
+        userId: 3,
         name: 'Test User 3',
         email: '[email protected]',
         joinedOrg: 'July 3, 2023',
       },
     },
     {
       enterpriseCustomerUser: {
-        user_id: 4,
+        userId: 4,
         name: 'Test User 4',
         email: '[email protected]',
         joinedOrg: 'July 4, 2024',