Skip to content

Commit d200b4c

Browse files
fix(Storage): completely remove usage filter (#1375)
1 parent 9e25733 commit d200b4c

File tree

12 files changed

+5
-268
lines changed

12 files changed

+5
-268
lines changed

src/containers/Storage/PaginatedStorage.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,6 @@ export const PaginatedStorage = ({
113113
handleVisibleEntitiesChange={handleGroupVisibilityChange}
114114
nodesUptimeFilter={nodesUptimeFilter}
115115
handleNodesUptimeFilterChange={handleUptimeFilterChange}
116-
withGroupsUsageFilter={false}
117116
entitiesCountCurrent={foundEntities}
118117
entitiesCountTotal={totalEntities}
119118
entitiesLoading={!inited}

src/containers/Storage/Storage.tsx

Lines changed: 3 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import React from 'react';
22

3-
import {ArrayParam, StringParam, useQueryParams, withDefault} from 'use-query-params';
3+
import {StringParam, useQueryParams} from 'use-query-params';
44

55
import {AccessDenied} from '../../components/Errors/403';
66
import {isAccessError} from '../../components/Errors/PageError/PageError';
@@ -13,11 +13,7 @@ import {
1313
import {useClusterBaseInfo} from '../../store/reducers/cluster/cluster';
1414
import type {NodesSortParams} from '../../store/reducers/nodes/types';
1515
import {VISIBLE_ENTITIES} from '../../store/reducers/storage/constants';
16-
import {
17-
filterGroups,
18-
filterNodes,
19-
getUsageFilterOptions,
20-
} from '../../store/reducers/storage/selectors';
16+
import {filterGroups, filterNodes} from '../../store/reducers/storage/selectors';
2117
import {storageApi} from '../../store/reducers/storage/storage';
2218
import {storageTypeSchema, visibleEntitiesSchema} from '../../store/reducers/storage/types';
2319
import type {
@@ -40,23 +36,6 @@ import {defaultSortNode, getDefaultSortGroup} from './utils';
4036

4137
import './Storage.scss';
4238

43-
const UsageFilterParam = withDefault(
44-
{
45-
encode: ArrayParam.encode,
46-
decode: (input) => {
47-
if (input === null || input === undefined) {
48-
return input;
49-
}
50-
51-
if (!Array.isArray(input)) {
52-
return input ? [input] : [];
53-
}
54-
return input.filter(Boolean) as string[];
55-
},
56-
},
57-
[],
58-
);
59-
6039
interface StorageProps {
6140
database?: string;
6241
nodeId?: string | number;
@@ -77,7 +56,6 @@ export const Storage = ({database, nodeId, groupId, pDiskId}: StorageProps) => {
7756
visible: StringParam,
7857
search: StringParam,
7958
uptimeFilter: StringParam,
80-
usageFilter: UsageFilterParam,
8159
});
8260
const storageType = storageTypeSchema.parse(queryParams.type);
8361
const isGroups = storageType === 'groups';
@@ -86,7 +64,6 @@ export const Storage = ({database, nodeId, groupId, pDiskId}: StorageProps) => {
8664
const visibleEntities = visibleEntitiesSchema.parse(queryParams.visible);
8765
const filter = queryParams.search ?? '';
8866
const uptimeFilter = nodesUptimeFilterValuesSchema.parse(queryParams.uptimeFilter);
89-
const usageFilter = queryParams.usageFilter;
9067

9168
const [nodeSort, setNodeSort] = React.useState<NodesSortParams>({
9269
sortOrder: undefined,
@@ -157,12 +134,7 @@ export const Storage = ({database, nodeId, groupId, pDiskId}: StorageProps) => {
157134
() => filterNodes(nodes, filter, uptimeFilter),
158135
[filter, nodes, uptimeFilter],
159136
);
160-
const storageGroups = React.useMemo(
161-
() => filterGroups(groups, filter, usageFilter),
162-
[filter, groups, usageFilter],
163-
);
164-
165-
const usageFilterOptions = React.useMemo(() => getUsageFilterOptions(groups), [groups]);
137+
const storageGroups = React.useMemo(() => filterGroups(groups, filter), [filter, groups]);
166138

167139
const [nodesSort, handleNodesSort] = useTableSort(nodesSortParams, (params) =>
168140
setNodeSort(params as NodesSortParams),
@@ -171,10 +143,6 @@ export const Storage = ({database, nodeId, groupId, pDiskId}: StorageProps) => {
171143
setGroupSort(params as StorageSortParams),
172144
);
173145

174-
const handleUsageFilterChange = (value: string[]) => {
175-
setQueryParams({usageFilter: value.length ? value : undefined}, 'replaceIn');
176-
};
177-
178146
const handleTextFilterChange = (value: string) => {
179147
setQueryParams({search: value || undefined}, 'replaceIn');
180148
};
@@ -248,9 +216,6 @@ export const Storage = ({database, nodeId, groupId, pDiskId}: StorageProps) => {
248216
handleVisibleEntitiesChange={handleGroupVisibilityChange}
249217
nodesUptimeFilter={uptimeFilter}
250218
handleNodesUptimeFilterChange={handleUptimeFilterChange}
251-
groupsUsageFilter={usageFilter}
252-
groupsUsageFilterOptions={usageFilterOptions}
253-
handleGroupsUsageFilterChange={handleUsageFilterChange}
254219
entitiesCountCurrent={entitiesCountCurrent}
255220
entitiesCountTotal={entitiesCount.total}
256221
entitiesLoading={isLoading}

src/containers/Storage/StorageControls/StorageControls.tsx

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,6 @@ import type {StorageType, VisibleEntities} from '../../../store/reducers/storage
1111
import type {NodesUptimeFilterValues} from '../../../utils/nodes';
1212
import {StorageTypeFilter} from '../StorageTypeFilter/StorageTypeFilter';
1313
import {StorageVisibleEntitiesFilter} from '../StorageVisibleEntitiesFilter/StorageVisibleEntitiesFilter';
14-
import {UsageFilter} from '../UsageFilter/UsageFilter';
15-
import type {UsageFilterItem} from '../UsageFilter/UsageFilter';
1614
import i18n from '../i18n';
1715
import {b} from '../shared';
1816

@@ -30,11 +28,6 @@ interface StorageControlsProps {
3028
nodesUptimeFilter: NodesUptimeFilterValues;
3129
handleNodesUptimeFilterChange: (value: NodesUptimeFilterValues) => void;
3230

33-
withGroupsUsageFilter?: boolean;
34-
groupsUsageFilter?: string[];
35-
groupsUsageFilterOptions?: UsageFilterItem[];
36-
handleGroupsUsageFilterChange?: (value: string[]) => void;
37-
3831
entitiesCountCurrent: number;
3932
entitiesCountTotal?: number;
4033
entitiesLoading: boolean;
@@ -57,11 +50,6 @@ export const StorageControls = ({
5750
nodesUptimeFilter,
5851
handleNodesUptimeFilterChange,
5952

60-
withGroupsUsageFilter,
61-
groupsUsageFilter,
62-
groupsUsageFilterOptions,
63-
handleGroupsUsageFilterChange,
64-
6553
entitiesCountCurrent,
6654
entitiesCountTotal,
6755
entitiesLoading,
@@ -70,7 +58,6 @@ export const StorageControls = ({
7058
handleSelectedColumnsUpdate,
7159
}: StorageControlsProps) => {
7260
const isNodes = storageType === STORAGE_TYPES.nodes;
73-
const isGroups = storageType === STORAGE_TYPES.groups;
7461
const entityName = isNodes ? i18n('nodes') : i18n('groups');
7562

7663
return (
@@ -96,14 +83,6 @@ export const StorageControls = ({
9683
{isNodes && (
9784
<UptimeFilter value={nodesUptimeFilter} onChange={handleNodesUptimeFilterChange} />
9885
)}
99-
{isGroups && withGroupsUsageFilter && (
100-
<UsageFilter
101-
value={groupsUsageFilter}
102-
onChange={handleGroupsUsageFilterChange}
103-
groups={groupsUsageFilterOptions}
104-
/>
105-
)}
106-
10786
<EntitiesCount
10887
label={entityName}
10988
loading={entitiesLoading}

src/containers/Storage/UsageFilter/UsageFilter.scss

Lines changed: 0 additions & 38 deletions
This file was deleted.

src/containers/Storage/UsageFilter/UsageFilter.tsx

Lines changed: 0 additions & 100 deletions
This file was deleted.

src/containers/Storage/UsageFilter/i18n/en.json

Lines changed: 0 additions & 5 deletions
This file was deleted.

src/containers/Storage/UsageFilter/i18n/index.ts

Lines changed: 0 additions & 8 deletions
This file was deleted.

src/containers/Storage/UsageFilter/i18n/ru.json

Lines changed: 0 additions & 5 deletions
This file was deleted.

src/containers/Storage/UsageFilter/index.ts

Lines changed: 0 additions & 1 deletion
This file was deleted.

src/containers/Storage/utils/index.ts

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ import type {
77
StorageSortParams,
88
VisibleEntities,
99
} from '../../../store/reducers/storage/types';
10-
import {EFlag} from '../../../types/api/enums';
1110
import {generateEvaluator} from '../../../utils/generateEvaluator';
1211
import {NODES_COLUMNS_IDS} from '../../Nodes/columns/constants';
1312
import {STORAGE_GROUPS_COLUMNS_IDS} from '../StorageGroups/columns/constants';
@@ -35,11 +34,6 @@ export const getUsageSeverityForStorageGroup = generateEvaluator(80, 85, [
3534
'warning',
3635
'danger',
3736
]);
38-
export const getUsageSeverityForEntityStatus = generateEvaluator(80, 85, [
39-
EFlag.Green,
40-
EFlag.Yellow,
41-
EFlag.Red,
42-
]);
4337

4438
export const defaultSortNode: NodesSortParams = {
4539
sortValue: NODES_COLUMNS_IDS.NodeId,

0 commit comments

Comments
 (0)