Remove unneeded params from hook.

elastic · Jun 5, 2020 · 518410d · 518410d
1 parent b7053b8
commit 518410d
Showing 1 changed file with 3 additions and 8 deletions.
diff --git a/x-pack/plugins/uptime/public/hooks/use_filter_update.ts b/x-pack/plugins/uptime/public/hooks/use_filter_update.ts
@@ -20,19 +20,14 @@ interface SelectedFilters {
   selectedFilters: Map<string, string[]>;
 }
 
-export const useFilterUpdate = (
-  fieldName?: string,
-  values?: string[],
-  shouldUpdateUrl?: boolean,
-  alertParamFilters?: any
-): SelectedFilters => {
+export const useFilterUpdate = (fieldName?: string, values?: string[]): SelectedFilters => {
   const [getUrlParams, updateUrl] = useUrlParams();
 
   const { filters: currentFilters } = getUrlParams();
 
   // update filters in the URL from filter group
   const onFilterUpdate = (filtersKuery: string) => {
-    if (currentFilters !== filtersKuery && shouldUpdateUrl) {
+    if (currentFilters !== filtersKuery) {
       updateUrl({ filters: filtersKuery, pagination: '' });
     }
   };
@@ -62,7 +57,7 @@ export const useFilterUpdate = (
     }
 
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [alertParamFilters, fieldName, values]);
+  }, [fieldName, values]);
 
   return {
     selectedTags: filterKueries.get('tags') || [],