From 21767b08caf2da9442323d103b6b8ee536832c73 Mon Sep 17 00:00:00 2001
From: Ainhoa Larumbe <ainhoal@spotify.com>
Date: Wed, 15 Sep 2021 13:55:23 +0100
Subject: [PATCH 1/3] core-components: Remove CheckboxTree component (used in
 Table Filters)

Signed-off-by: Ainhoa Larumbe <ainhoal@spotify.com>
---
 .changeset/red-hats-walk.md                   |   8 +
 .../CheckboxTree/CheckboxTree.stories.tsx     | 105 ------
 .../CheckboxTree/CheckboxTree.test.tsx        |  59 ---
 .../components/CheckboxTree/CheckboxTree.tsx  | 357 ------------------
 .../src/components/CheckboxTree/index.tsx     |  17 -
 .../src/components/Table/Filters.tsx          |  73 +---
 .../src/components/Table/Table.stories.tsx    |   4 -
 .../src/components/Table/Table.tsx            |  18 +-
 8 files changed, 26 insertions(+), 615 deletions(-)
 create mode 100644 .changeset/red-hats-walk.md
 delete mode 100644 packages/core-components/src/components/CheckboxTree/CheckboxTree.stories.tsx
 delete mode 100644 packages/core-components/src/components/CheckboxTree/CheckboxTree.test.tsx
 delete mode 100644 packages/core-components/src/components/CheckboxTree/CheckboxTree.tsx
 delete mode 100644 packages/core-components/src/components/CheckboxTree/index.tsx

diff --git a/.changeset/red-hats-walk.md b/.changeset/red-hats-walk.md
new file mode 100644
index 0000000000..2d6eb06666
--- /dev/null
+++ b/.changeset/red-hats-walk.md
@@ -0,0 +1,8 @@
+---
+'@backstage/core-components': minor
+---
+
+Checkbox tree filters are no longer available in the Table component:
+
+- Deleted the `CheckboxTree` component
+- Removed the filter type `'checkbox-tree'` from the `TableFilter` types.
diff --git a/packages/core-components/src/components/CheckboxTree/CheckboxTree.stories.tsx b/packages/core-components/src/components/CheckboxTree/CheckboxTree.stories.tsx
deleted file mode 100644
index 48da8b92b2..0000000000
--- a/packages/core-components/src/components/CheckboxTree/CheckboxTree.stories.tsx
+++ /dev/null
@@ -1,105 +0,0 @@
-/*
- * Copyright 2020 The Backstage Authors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-import React, { useState } from 'react';
-import { CheckboxTree } from './CheckboxTree';
-
-const CHECKBOX_TREE_ITEMS = [
-  {
-    label: 'Generic subcategory name 1',
-    options: [
-      {
-        label: 'Option 1',
-        value: 1,
-      },
-      {
-        label: 'Option 2',
-        value: 2,
-      },
-    ],
-  },
-  {
-    label: 'Generic subcategory name 2',
-    options: [
-      {
-        label: 'Option 1',
-        value: 1,
-      },
-      {
-        label: 'Option 2',
-        value: 2,
-      },
-    ],
-  },
-  {
-    label: 'Generic subcategory name 3',
-    options: [
-      {
-        label: 'Option 1',
-        value: 1,
-      },
-      {
-        label: 'Option 2',
-        value: 2,
-      },
-    ],
-  },
-];
-
-export default {
-  title: 'Inputs/CheckboxTree',
-  component: CheckboxTree,
-};
-
-export const Default = () => (
-  <CheckboxTree
-    onChange={() => {}}
-    label="default"
-    subCategories={CHECKBOX_TREE_ITEMS}
-  />
-);
-
-export const DynamicTree = () => {
-  function generateTree(showMore: boolean = false) {
-    const t = [
-      {
-        label: 'Show more',
-        options: [],
-      },
-    ];
-
-    if (showMore) {
-      t.push({
-        label: 'More',
-        options: [],
-      });
-    }
-
-    return t;
-  }
-
-  const [tree, setTree] = useState(generateTree());
-
-  return (
-    <CheckboxTree
-      onChange={state => {
-        setTree(generateTree(state.some(c => c.category === 'Show more')));
-      }}
-      label="default"
-      subCategories={tree}
-    />
-  );
-};
diff --git a/packages/core-components/src/components/CheckboxTree/CheckboxTree.test.tsx b/packages/core-components/src/components/CheckboxTree/CheckboxTree.test.tsx
deleted file mode 100644
index 6c84a311d2..0000000000
--- a/packages/core-components/src/components/CheckboxTree/CheckboxTree.test.tsx
+++ /dev/null
@@ -1,59 +0,0 @@
-/*
- * Copyright 2020 The Backstage Authors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-import { fireEvent, render } from '@testing-library/react';
-import React from 'react';
-import { CheckboxTree } from './CheckboxTree';
-
-const CHECKBOX_TREE_ITEMS = [
-  {
-    label: 'Generic subcategory name 1',
-    options: [
-      {
-        label: 'Option 1',
-        value: 1,
-      },
-      {
-        label: 'Option 2',
-        value: 2,
-      },
-    ],
-  },
-];
-
-const minProps = {
-  onChange: jest.fn(),
-  label: 'Default',
-  subCategories: CHECKBOX_TREE_ITEMS,
-};
-
-describe('<CheckboxTree />', () => {
-  it('renders without exploding', async () => {
-    const { getByText, getByTestId } = render(<CheckboxTree {...minProps} />);
-
-    expect(getByText('Generic subcategory name 1')).toBeInTheDocument();
-    const checkbox = await getByTestId('expandable');
-
-    // Simulate click on expandable arrow
-    fireEvent.click(checkbox);
-
-    // Simulate click on option
-    const option = getByText('Option 1');
-    expect(getByText('Option 1')).toBeInTheDocument();
-    fireEvent.click(option);
-    expect(minProps.onChange).toHaveBeenCalled();
-  });
-});
diff --git a/packages/core-components/src/components/CheckboxTree/CheckboxTree.tsx b/packages/core-components/src/components/CheckboxTree/CheckboxTree.tsx
deleted file mode 100644
index 71b5cf7659..0000000000
--- a/packages/core-components/src/components/CheckboxTree/CheckboxTree.tsx
+++ /dev/null
@@ -1,357 +0,0 @@
-/*
- * Copyright 2020 The Backstage Authors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-/* eslint-disable guard-for-in */
-import {
-  Checkbox,
-  Collapse,
-  List,
-  ListItem,
-  ListItemIcon,
-  ListItemText,
-  Typography,
-} from '@material-ui/core';
-import { createStyles, makeStyles, Theme } from '@material-ui/core/styles';
-import ExpandLess from '@material-ui/icons/ExpandLess';
-import ExpandMore from '@material-ui/icons/ExpandMore';
-import produce from 'immer';
-import { isEqual } from 'lodash';
-import React, { useEffect, useReducer } from 'react';
-import { usePrevious } from 'react-use';
-
-type IndexedObject<T> = {
-  [key: string]: T;
-};
-
-const useStyles = makeStyles((theme: Theme) =>
-  createStyles({
-    root: {
-      width: '100%',
-      minWidth: 10,
-      maxWidth: 360,
-      backgroundColor: 'transparent',
-      '&:hover': {
-        backgroundColor: 'transparent',
-      },
-      '&:active': {
-        animation: 'none',
-        transform: 'none',
-      },
-    },
-    nested: {
-      paddingLeft: theme.spacing(5),
-      height: '32px',
-      '&:hover': {
-        backgroundColor: 'transparent',
-      },
-    },
-    listItemIcon: {
-      minWidth: 10,
-    },
-    listItem: {
-      '&:hover': {
-        backgroundColor: 'transparent',
-      },
-    },
-    text: {
-      '& span, & svg': {
-        fontWeight: 'normal',
-        fontSize: 14,
-      },
-    },
-  }),
-);
-
-/* SUB_CATEGORY */
-
-type SubCategory = {
-  label: string;
-  isChecked?: boolean;
-  isOpen?: boolean;
-  options?: Option[];
-};
-
-type SubCategoryWithIndexedOptions = {
-  label: string;
-  isChecked?: boolean;
-  isOpen?: boolean;
-  options: IndexedObject<Option>;
-};
-
-/* OPTION */
-
-type Option = {
-  label: string;
-  value: string | number;
-  isChecked?: boolean;
-};
-
-type Selection = { category?: string; selectedChildren?: string[] }[];
-
-export type CheckboxTreeProps = {
-  subCategories: SubCategory[];
-  label: string;
-  triggerReset?: boolean;
-  selected?: Selection;
-  onChange: (arg: Selection) => any;
-};
-
-/* REDUCER */
-
-type checkOptionPayload = {
-  subCategoryLabel: string;
-  optionLabel: string;
-};
-
-type Action =
-  | { type: 'checkOption'; payload: checkOptionPayload }
-  | { type: 'checkCategory'; payload: string }
-  | { type: 'toggleCategory'; payload: string }
-  | {
-      type: 'updateCategories';
-      payload: IndexedObject<SubCategoryWithIndexedOptions>;
-    }
-  | { type: 'updateSelected'; payload: Selection }
-  | { type: 'triggerReset' };
-
-const reducer = (
-  state: IndexedObject<SubCategoryWithIndexedOptions>,
-  action: Action,
-) => {
-  switch (action.type) {
-    case 'checkOption': {
-      return produce(state, newState => {
-        const category = newState[action.payload.subCategoryLabel];
-        const option = category.options[action.payload.optionLabel];
-        option.isChecked = !option.isChecked;
-        category.isChecked = Object.values(category.options).every(
-          o => o.isChecked,
-        );
-      });
-    }
-    case 'checkCategory': {
-      return produce(state, newState => {
-        const category = newState[action.payload];
-        const options = category.options;
-        category.isChecked = !category.isChecked;
-        for (const option in options) {
-          options[option].isChecked = category.isChecked;
-        }
-      });
-    }
-    case 'toggleCategory':
-      return produce(state, newState => {
-        const category = newState[action.payload];
-        category.isOpen = !category.isOpen;
-      });
-    case 'triggerReset': {
-      return produce(state, newState => {
-        for (const category in newState) {
-          newState[category].isChecked = false;
-          for (const option in newState[category].options) {
-            newState[category].options[option].isChecked =
-              newState[category].isChecked;
-          }
-        }
-      });
-    }
-    case 'updateCategories': {
-      return produce(state, newState => {
-        for (const category in newState) {
-          delete newState[category];
-        }
-
-        for (const category in action.payload) {
-          newState[category] = action.payload[category];
-
-          if (state[category]) {
-            newState[category].isChecked = state[category].isChecked;
-            newState[category].isOpen = state[category].isOpen;
-          }
-        }
-      });
-    }
-    case 'updateSelected': {
-      return produce(state, newState => {
-        for (const category in newState) {
-          const selection = action.payload.find(s => s.category === category);
-
-          if (selection) {
-            newState[category].isChecked = true;
-
-            for (const option in newState[category].options) {
-              newState[category].options[option].isChecked =
-                selection.selectedChildren?.includes(option) || false;
-            }
-          }
-        }
-      });
-    }
-    default:
-      return state;
-  }
-};
-
-const indexer = (
-  arr: SubCategory[],
-): IndexedObject<SubCategoryWithIndexedOptions> =>
-  arr.reduce((accumulator, el) => {
-    if (el.options) {
-      return {
-        ...accumulator,
-        [el.label]: {
-          label: el.label,
-          isChecked: el.isChecked || false,
-          isOpen: false,
-          options: indexer(el.options),
-        },
-      };
-    }
-    return {
-      ...accumulator,
-      [el.label]: { ...el, isChecked: el.isChecked || false },
-    };
-  }, {});
-
-/**
- *
- * @deprecated CheckboxTree is no longer used in Table filters
- */
-export function CheckboxTree(props: CheckboxTreeProps) {
-  const { subCategories, label, selected, onChange, triggerReset } = props;
-  const classes = useStyles();
-
-  const [state, dispatch] = useReducer(reducer, indexer(subCategories));
-
-  const handleOpen = (event: any, value: any) => {
-    event.stopPropagation();
-    dispatch({ type: 'toggleCategory', payload: value });
-  };
-
-  const previousSubCategories = usePrevious(subCategories);
-
-  useEffect(() => {
-    const values = Object.values(state).map(category => ({
-      category: category.isChecked ? category.label : undefined,
-      selectedChildren: Object.values(category.options)
-        .filter(option => option.isChecked)
-        .map(option => option.label),
-    }));
-    onChange(values);
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [state]);
-
-  useEffect(() => {
-    dispatch({ type: 'triggerReset' });
-  }, [triggerReset]);
-
-  useEffect(() => {
-    if (selected) {
-      dispatch({ type: 'updateSelected', payload: selected });
-    }
-  }, [selected]);
-
-  useEffect(() => {
-    if (!isEqual(subCategories, previousSubCategories)) {
-      dispatch({
-        type: 'updateCategories',
-        payload: indexer(subCategories),
-      });
-    }
-  }, [subCategories, previousSubCategories]);
-
-  return (
-    <div>
-      <Typography variant="button">{label}</Typography>
-      <List className={classes.root}>
-        {Object.values(state).map(item => (
-          <div key={item.label}>
-            <ListItem
-              className={classes.listItem}
-              dense
-              button
-              onClick={() =>
-                dispatch({
-                  type: 'checkCategory',
-                  payload: item.label,
-                })
-              }
-            >
-              <ListItemIcon className={classes.listItemIcon}>
-                <Checkbox
-                  color="primary"
-                  edge="start"
-                  checked={item.isChecked}
-                  tabIndex={-1}
-                  disableRipple
-                />
-              </ListItemIcon>
-              <ListItemText className={classes.text} primary={item.label} />
-              {Object.values(item.options).length ? (
-                <>
-                  {item.isOpen ? (
-                    <ExpandLess
-                      data-testid="expandable"
-                      onClick={event => handleOpen(event, item.label)}
-                    />
-                  ) : (
-                    <ExpandMore
-                      data-testid="expandable"
-                      onClick={event => handleOpen(event, item.label)}
-                    />
-                  )}
-                </>
-              ) : null}
-            </ListItem>
-            <Collapse in={item.isOpen} timeout="auto" unmountOnExit>
-              <List component="div" disablePadding>
-                {Object.values(item.options).map(option => (
-                  <ListItem
-                    button
-                    key={option.label}
-                    className={classes.nested}
-                    onClick={() =>
-                      dispatch({
-                        type: 'checkOption',
-                        payload: {
-                          subCategoryLabel: item.label,
-                          optionLabel: option.label,
-                        },
-                      })
-                    }
-                  >
-                    <ListItemIcon className={classes.listItemIcon}>
-                      <Checkbox
-                        color="primary"
-                        edge="start"
-                        checked={option.isChecked}
-                        tabIndex={-1}
-                        disableRipple
-                      />
-                    </ListItemIcon>
-                    <ListItemText
-                      className={classes.text}
-                      primary={option.label}
-                    />
-                  </ListItem>
-                ))}
-              </List>
-            </Collapse>
-          </div>
-        ))}
-      </List>
-    </div>
-  );
-}
diff --git a/packages/core-components/src/components/CheckboxTree/index.tsx b/packages/core-components/src/components/CheckboxTree/index.tsx
deleted file mode 100644
index e1ceb98ee4..0000000000
--- a/packages/core-components/src/components/CheckboxTree/index.tsx
+++ /dev/null
@@ -1,17 +0,0 @@
-/*
- * Copyright 2020 The Backstage Authors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-export { CheckboxTree } from './CheckboxTree';
diff --git a/packages/core-components/src/components/Table/Filters.tsx b/packages/core-components/src/components/Table/Filters.tsx
index b482e6c808..bd70e3eb64 100644
--- a/packages/core-components/src/components/Table/Filters.tsx
+++ b/packages/core-components/src/components/Table/Filters.tsx
@@ -18,8 +18,6 @@ import React, { useEffect, useState } from 'react';
 import { BackstageTheme } from '@backstage/theme';
 import { Button, makeStyles } from '@material-ui/core';
 import { Select } from '../Select';
-import { CheckboxTree } from '../CheckboxTree';
-import { CheckboxTreeProps } from '../CheckboxTree/CheckboxTree';
 import { SelectProps } from '../Select/Select';
 
 const useSubvalueCellStyles = makeStyles<BackstageTheme>(theme => ({
@@ -53,10 +51,8 @@ const useSubvalueCellStyles = makeStyles<BackstageTheme>(theme => ({
 export type Without<T, K> = Pick<T, Exclude<keyof T, K>>;
 
 export type Filter = {
-  type: 'select' | /** @deprecated */ 'checkbox-tree' | 'multiple-select';
-  element:
-    | Without<CheckboxTreeProps, 'onChange'>
-    | Without<SelectProps, 'onChange'>;
+  type: 'select' | 'multiple-select';
+  element: Without<SelectProps, 'onChange'>;
 };
 
 export type SelectedFilters = {
@@ -100,57 +96,20 @@ export const Filters = (props: Props) => {
       </div>
       <div className={classes.filters}>
         {props.filters?.length &&
-          props.filters.map(filter =>
-            filter.type === 'checkbox-tree' ? (
-              <CheckboxTree
-                triggerReset={reset}
-                key={filter.element.label}
-                {...(filter.element as CheckboxTreeProps)}
-                selected={
-                  selectedFilters[filter.element.label]
-                    ? (selectedFilters[filter.element.label] as string[]).map(
-                        s => ({
-                          category: s,
-                        }),
-                      )
-                    : undefined
-                }
-                onChange={el =>
-                  setSelectedFilters({
-                    ...selectedFilters,
-                    [filter.element.label]: el
-                      .filter(
-                        (checkboxFilter: any) =>
-                          checkboxFilter.category ||
-                          checkboxFilter.selectedChildren.length,
-                      )
-                      .map((checkboxFilter: any) =>
-                        checkboxFilter.category
-                          ? [
-                              ...checkboxFilter.selectedChildren,
-                              checkboxFilter.category,
-                            ]
-                          : checkboxFilter.selectedChildren,
-                      )
-                      .flat(),
-                  })
-                }
-              />
-            ) : (
-              <Select
-                triggerReset={reset}
-                key={filter.element.label}
-                {...(filter.element as SelectProps)}
-                selected={selectedFilters[filter.element.label]}
-                onChange={el =>
-                  setSelectedFilters({
-                    ...selectedFilters,
-                    [filter.element.label]: el as any,
-                  })
-                }
-              />
-            ),
-          )}
+          props.filters.map(filter => (
+            <Select
+              triggerReset={reset}
+              key={filter.element.label}
+              {...(filter.element as SelectProps)}
+              selected={selectedFilters[filter.element.label]}
+              onChange={el =>
+                setSelectedFilters({
+                  ...selectedFilters,
+                  [filter.element.label]: el as any,
+                })
+              }
+            />
+          ))}
       </div>
     </div>
   );
diff --git a/packages/core-components/src/components/Table/Table.stories.tsx b/packages/core-components/src/components/Table/Table.stories.tsx
index fd05235096..27fd33f32c 100644
--- a/packages/core-components/src/components/Table/Table.stories.tsx
+++ b/packages/core-components/src/components/Table/Table.stories.tsx
@@ -313,10 +313,6 @@ export const FilterTable = () => {
       column: 'Column 2',
       type: 'multiple-select',
     },
-    {
-      column: 'Numeric value',
-      type: 'checkbox-tree',
-    },
   ];
 
   return (
diff --git a/packages/core-components/src/components/Table/Table.tsx b/packages/core-components/src/components/Table/Table.tsx
index ecde7e992b..163d8392c8 100644
--- a/packages/core-components/src/components/Table/Table.tsx
+++ b/packages/core-components/src/components/Table/Table.tsx
@@ -56,7 +56,6 @@ import React, {
   useEffect,
   useState,
 } from 'react';
-import { CheckboxTreeProps } from '../CheckboxTree/CheckboxTree';
 import { SelectProps } from '../Select/Select';
 import { Filter, Filters, SelectedFilters, Without } from './Filters';
 
@@ -191,7 +190,7 @@ export interface TableColumn<T extends object = {}> extends Column<T> {
 
 export type TableFilter = {
   column: string;
-  type: 'select' | 'multiple-select' | /** @deprecated */ 'checkbox-tree';
+  type: 'select' | 'multiple-select';
 };
 
 export type TableState = {
@@ -403,16 +402,6 @@ export function Table<T extends object = {}>(props: TableProps<T>) {
       return distinctValues;
     };
 
-    const constructCheckboxTree = (
-      filter: TableFilter,
-    ): Without<CheckboxTreeProps, 'onChange'> => ({
-      label: filter.column,
-      subCategories: [...extractDistinctValues(filter.column)].map(v => ({
-        label: v,
-        options: [],
-      })),
-    });
-
     const constructSelect = (
       filter: TableFilter,
     ): Without<SelectProps, 'onChange'> => {
@@ -429,10 +418,7 @@ export function Table<T extends object = {}>(props: TableProps<T>) {
 
     return filterConfig.map(filter => ({
       type: filter.type,
-      element:
-        filter.type === 'checkbox-tree'
-          ? constructCheckboxTree(filter)
-          : constructSelect(filter),
+      element: constructSelect(filter),
     }));
   };
 

From 7837adbbb84a9511354e716f69e5f018813f0f93 Mon Sep 17 00:00:00 2001
From: Ainhoa Larumbe <ainhoal@spotify.com>
Date: Wed, 15 Sep 2021 14:38:38 +0100
Subject: [PATCH 2/3] update api-report

Signed-off-by: Ainhoa Larumbe <ainhoal@spotify.com>
---
 packages/core-components/api-report.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/core-components/api-report.md b/packages/core-components/api-report.md
index e987d6b67e..cb08c3424d 100644
--- a/packages/core-components/api-report.md
+++ b/packages/core-components/api-report.md
@@ -1889,7 +1889,7 @@ export interface TableColumn<T extends object = {}> extends Column<T> {
 // @public (undocumented)
 export type TableFilter = {
   column: string;
-  type: 'select' | 'multiple-select' | /** @deprecated */ 'checkbox-tree';
+  type: 'select' | 'multiple-select';
 };
 
 // Warning: (ae-missing-release-tag) "TableProps" is exported by the package, but it is missing a release tag (@alpha, @beta, @public, or @internal)

From fb3c2e8d019de0d2acb9659c77691f31c704003f Mon Sep 17 00:00:00 2001
From: Ainhoa Larumbe <ainhoal@spotify.com>
Date: Thu, 16 Sep 2021 13:43:08 +0100
Subject: [PATCH 3/3] remove check for checkbox-tree filter type

Signed-off-by: Ainhoa Larumbe <ainhoal@spotify.com>
---
 packages/core-components/src/components/Table/Table.tsx | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/packages/core-components/src/components/Table/Table.tsx b/packages/core-components/src/components/Table/Table.tsx
index 163d8392c8..627e5b8ccc 100644
--- a/packages/core-components/src/components/Table/Table.tsx
+++ b/packages/core-components/src/components/Table/Table.tsx
@@ -364,14 +364,6 @@ export function Table<T extends object = {}>(props: TableProps<T>) {
     setSelectedFiltersLength(selectedFiltersArray.flat().length);
   }, [data, selectedFilters, getFieldByTitle]);
 
-  // Check for deprecated checkbox-tree filter
-  useEffect(() => {
-    if (filters?.some(filter => filter.type === 'checkbox-tree')) {
-      // eslint-disable-next-line no-console
-      console.warn('"checkbox-tree" filter type is deprecated');
-    }
-  }, [filters]);
-
   const constructFilters = (
     filterConfig: TableFilter[],
     dataValue: any[] | undefined,