From 39f6eaf5f404154bb9eb86be00a672a2987929f3 Mon Sep 17 00:00:00 2001
From: Maryia Lapata <mary.lopato@gmail.com>
Date: Wed, 26 Feb 2020 13:20:24 +0300
Subject: [PATCH] Fix TS for vis_type_table (#58347)

* Remove eslint overrides for vis_type_table

* Fix TS for vis_type_table

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
---
 .eslintrc.js                                                | 6 ------
 .../vis_type_table/public/components/table_vis_options.tsx  | 6 +++---
 2 files changed, 3 insertions(+), 9 deletions(-)

diff --git a/.eslintrc.js b/.eslintrc.js
index 9b00135df5bac..087d6276cd33f 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -82,12 +82,6 @@ module.exports = {
         'react-hooks/exhaustive-deps': 'off',
       },
     },
-    {
-      files: ['src/legacy/core_plugins/vis_type_table/**/*.{js,ts,tsx}'],
-      rules: {
-        'react-hooks/exhaustive-deps': 'off',
-      },
-    },
     {
       files: [
         'src/legacy/core_plugins/vis_default_editor/public/components/controls/**/*.{ts,tsx}',
diff --git a/src/legacy/core_plugins/vis_type_table/public/components/table_vis_options.tsx b/src/legacy/core_plugins/vis_type_table/public/components/table_vis_options.tsx
index 5729618b6ae07..8cc0ca2456867 100644
--- a/src/legacy/core_plugins/vis_type_table/public/components/table_vis_options.tsx
+++ b/src/legacy/core_plugins/vis_type_table/public/components/table_vis_options.tsx
@@ -47,14 +47,14 @@ function TableOptions({
         .filter(col => get(col.aggConfig.type.getFormat(col.aggConfig), 'type.id') === 'number')
         .map(({ name }) => ({ value: name, text: name })),
     ],
-    [aggs, stateParams.percentageCol, stateParams.dimensions]
+    [aggs]
   );
 
   const isPerPageValid = stateParams.perPage === '' || stateParams.perPage > 0;
 
   useEffect(() => {
     setValidity(isPerPageValid);
-  }, [isPerPageValid]);
+  }, [isPerPageValid, setValidity]);
 
   useEffect(() => {
     if (
@@ -64,7 +64,7 @@ function TableOptions({
     ) {
       setValue('percentageCol', percentageColumns[0].value);
     }
-  }, [percentageColumns, stateParams.percentageCol]);
+  }, [percentageColumns, stateParams.percentageCol, setValidity, setValue]);
 
   return (
     <EuiPanel paddingSize="s">