Skip to content

Commit 9669463

Browse files
removing unused providers
1 parent 822c966 commit 9669463

File tree

3 files changed

+5
-17
lines changed

3 files changed

+5
-17
lines changed

src/components/Graphs/Graph.tsx

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ import {
1818
useProvidersConfigResource,
1919
} from '../../lib/api/useApiResource';
2020
import { ManagedResourcesRequest } from '../../lib/api/types/crossplane/listManagedResources';
21-
import { ProvidersListRequest } from '../../lib/api/types/crossplane/listProviders';
2221
import { resourcesInterval } from '../../lib/shared/constants';
2322
import { ThemingParameters } from '@ui5/webcomponents-react-base';
2423
import { NodeData, ManagedResourceGroup, ManagedResourceItem } from './types';
@@ -110,12 +109,6 @@ const Graph: React.FC = () => {
110109
useApiResource(ManagedResourcesRequest, {
111110
refreshInterval: resourcesInterval,
112111
});
113-
const { data: providers, error: providersError } = useApiResource(
114-
ProvidersListRequest,
115-
{
116-
refreshInterval: resourcesInterval,
117-
},
118-
);
119112
const { data: providerConfigsList, error: providerConfigsError } =
120113
useProvidersConfigResource({
121114
refreshInterval: resourcesInterval,
@@ -196,8 +189,8 @@ const Graph: React.FC = () => {
196189
}, [managedResources, providerConfigsList]);
197190

198191
const colorMap = useMemo(
199-
() => generateColorMap(treeData, colorBy, providers?.items),
200-
[treeData, colorBy, providers?.items],
192+
() => generateColorMap(treeData, colorBy),
193+
[treeData, colorBy],
201194
);
202195

203196
useEffect(() => {
@@ -207,15 +200,15 @@ const Graph: React.FC = () => {
207200
setEdges(edges);
208201
}, [treeData, colorBy, colorMap, setNodes, setEdges]);
209202

210-
if (managedResourcesError || providersError || providerConfigsError) {
203+
if (managedResourcesError || providerConfigsError) {
211204
return (
212205
<div className={`${styles.centeredMessage} ${styles.errorMessage}`}>
213206
Error loading graph data.
214207
</div>
215208
);
216209
}
217210

218-
if (!managedResources || !providers || !providerConfigsList) {
211+
if (!managedResources || !providerConfigsList) {
219212
return <div className={styles.centeredMessage}>Loading graph data...</div>;
220213
}
221214

@@ -290,7 +283,6 @@ const Graph: React.FC = () => {
290283
<Legend
291284
nodes={nodes.map((n) => n.data as NodeData)}
292285
colorBy={colorBy}
293-
providers={providers?.items || []}
294286
generateColorMap={generateColorMap}
295287
/>
296288
</div>

src/components/Graphs/Legend.tsx

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,17 +9,15 @@ interface LegendProps {
99
generateColorMap: (
1010
items: NodeData[],
1111
colorBy: string,
12-
providers?: ProviderConfigItem[],
1312
) => Record<string, string>;
1413
}
1514

1615
export const Legend: React.FC<LegendProps> = ({
1716
nodes,
1817
colorBy,
19-
providers,
2018
generateColorMap,
2119
}) => {
22-
const colorMap = generateColorMap(nodes, colorBy, providers);
20+
const colorMap = generateColorMap(nodes, colorBy);
2321
return (
2422
<div className={styles.legendContainer}>
2523
<h4 className={styles.legendTitle}>Legend:</h4>

src/components/Graphs/graphUtils.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ import {
33
ManagedResourceItem,
44
NodeData,
55
ProviderConfig,
6-
ProviderConfigItem,
76
} from './types';
87

98
export type StatusType = 'ERROR' | 'OK';
@@ -45,7 +44,6 @@ export const resolveProviderType = (
4544
export const generateColorMap = (
4645
items: NodeData[],
4746
colorBy: string,
48-
_providers?: ProviderConfigItem[],
4947
): Record<string, string> => {
5048
const colors = [
5149
'#1abc9c',

0 commit comments

Comments
 (0)