Skip to content

Commit b39fc53

Browse files
authored
Merge pull request #162 from docker/remove-unused-code
Remove unused code in App
2 parents cf7da5c + f3f3131 commit b39fc53

File tree

1 file changed

+10
-24
lines changed

1 file changed

+10
-24
lines changed

src/extension/ui/src/App.tsx

Lines changed: 10 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1,30 +1,18 @@
1-
import React, { useState, Suspense, useEffect } from 'react';
21
import { createDockerDesktopClient } from '@docker/extension-api-client';
3-
import { Typography, Button, IconButton, Alert, DialogTitle, Dialog, DialogContent, CircularProgress, Paper, Box, SvgIcon, useTheme } from '@mui/material';
4-
import { CatalogItemRichened } from './types/catalog';
5-
import { Close } from '@mui/icons-material';
2+
import { useEffect } from 'react';
3+
64
import { CatalogGrid } from './components/CatalogGrid';
7-
import { POLL_INTERVAL } from './Constants';
8-
import ConfigurationModal from './components/tile/Modal';
95
import LoadingState from './components/LoadingState';
106
import { useCatalogAll } from './queries/useCatalog';
11-
import { useRequiredImages } from './queries/useRequiredImages';
12-
import { useMCPClient } from './queries/useMCPClient';
137
import { useConfig } from './queries/useConfig';
8+
import { useMCPClient } from './queries/useMCPClient';
9+
import { useRequiredImages } from './queries/useRequiredImages';
1410
import { useSecrets } from './queries/useSecrets';
15-
import { syncConfigWithRegistry, syncRegistryWithConfig } from './Registry';
11+
import { syncRegistryWithConfig } from './Registry';
1612

1713
export const client = createDockerDesktopClient();
1814

19-
const DEFAULT_SETTINGS = {
20-
showModal: false,
21-
pollIntervalSeconds: POLL_INTERVAL / 1000
22-
}
23-
2415
export function App() {
25-
const [settings, setSettings] = useState<{ showModal: boolean, pollIntervalSeconds: number }>(localStorage.getItem('settings') ? JSON.parse(localStorage.getItem('settings') || '{}') : DEFAULT_SETTINGS);
26-
const [configuringItem, setConfiguringItem] = useState<CatalogItemRichened | null>(null);
27-
2816
// Use hooks directly in the component
2917
const catalogAll = useCatalogAll(client);
3018
const requiredImages = useRequiredImages(client);
@@ -44,17 +32,17 @@ export function App() {
4432
...mcpClient,
4533

4634
// Config props
47-
...config
35+
...config,
4836
};
4937

50-
const isLoading = catalogAll.catalogLoading ||
38+
const isLoading =
39+
catalogAll.catalogLoading ||
5140
catalogAll.registryLoading ||
5241
requiredImages.isLoading ||
5342
secrets.isLoading;
5443

5544
useEffect(() => {
5645
if (config.config && catalogAll.registryItems) {
57-
console.log('registryItems', catalogAll.registryItems)
5846
syncRegistryWithConfig(client, catalogAll.registryItems, config.config);
5947
}
6048
}, [config.config]);
@@ -64,10 +52,8 @@ export function App() {
6452
{isLoading ? (
6553
<LoadingState appProps={appProps} />
6654
) : (
67-
<CatalogGrid
68-
appProps={appProps}
69-
/>
55+
<CatalogGrid appProps={appProps} />
7056
)}
7157
</>
7258
);
73-
}
59+
}

0 commit comments

Comments
 (0)