Merge pull request #588 from jbetancur/fix/558-linting

fixes linting issues and useMemo props
jbetancur · May 28, 2020 · 44c82db · 44c82db
2 parents fc69b51 + 05f2974
commit 44c82db
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/DataTable/DataTable.js b/src/DataTable/DataTable.js
@@ -188,7 +188,7 @@ const DataTable = memo(({
   }, [paginationTotalRows]);
 
   const columnsBySelector = useMemo(() => {
-    return columns.reduce((acc, item) => ({...acc, [item.selector]: item}), {})
+    return columns.reduce((acc, item) => ({ ...acc, [item.selector]: item }), {});
   }, [columns]);
 
   const sortedData = useMemo(() => {
@@ -209,7 +209,7 @@ const DataTable = memo(({
       : (a, b) => column.sortFunction(a, b) * -1;
 
     return [...data].sort(customSortFunction);
-  }, [data, columns, sortColumn, sortDirection, sortFunction, sortServer]);
+  }, [sortServer, sortColumn, columnsBySelector, sortDirection, data, sortFunction]);
 
   const calculatedRows = useMemo(() => {
     if (pagination && !paginationServer) {