chore: deleted unneeded comments

openedx · Jan 14, 2025 · 6fa63cb · 6fa63cb
1 parent 31f3350
commit 6fa63cb
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 11 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);