fix: localColumns reset when resizableColumns length = 0

hemengke1997 · Jan 11, 2022 · 7e255cb · 7e255cb
1 parent 32e68ee
commit 7e255cb
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/src/utils/useLocalColumns.ts b/src/utils/useLocalColumns.ts
@@ -56,11 +56,7 @@ function useLocalColumns<T extends ColumnOriginType<T>>({
   const [localColumns, setLocalColumns] = React.useState<T[] | undefined>(initLocalColumns);
 
   useEffect(() => {
-    if (!localColumns?.length) {
-      setLocalColumns(columnsProp);
-    } else {
-      setLocalColumns(initLocalColumns());
-    }
+    setLocalColumns(initLocalColumns());
     // eslint-disable-next-line react-hooks/exhaustive-deps
   }, [columnsProp]);
 
@@ -70,7 +66,7 @@ function useLocalColumns<T extends ColumnOriginType<T>>({
   React.useEffect(() => {
     const { persistenceType, persistenceKey } = columnsState || {};
 
-    if (!persistenceKey || !persistenceType || !resizableColumns) {
+    if (!persistenceKey || !persistenceType || !resizableColumns?.length) {
       return;
     }
     if (typeof window === 'undefined') return;
@@ -92,7 +88,8 @@ function useLocalColumns<T extends ColumnOriginType<T>>({
     } catch (error) {
       console.error(error);
     }
-  }, [columnsState, resizableColumns]);
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [resizableColumns]);
 
   /**
    * reset