Skip to content

Commit

Permalink
chore(settings): remove docker settings page (#362)
Browse files Browse the repository at this point in the history
Description
---
Remove docker settings page

Motivation and Context
---
It doesn't work anyway

How Has This Been Tested?
---
Manually
  • Loading branch information
mmrrnn authored Jul 5, 2024
1 parent 628a762 commit ac970ab
Show file tree
Hide file tree
Showing 7 changed files with 0 additions and 221 deletions.

This file was deleted.

This file was deleted.

11 changes: 0 additions & 11 deletions gui-react/src/containers/SettingsContainer/SettingsComponent.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ import {
} from './styles'
import BaseNodeSettings from './BaseNodeSettings'
import MiningSettings from './MiningSettings'
import DockerSettings from './DockerSettings'
import WalletSettings from './WalletSettings'
import ResetSettings from './ResetSettings'
import {
Expand Down Expand Up @@ -64,16 +63,6 @@ const renderSettings = (
setValue={props.setValue}
/>
)
case Settings.Docker:
return (
<DockerSettings
formState={props.formState}
control={props.control}
values={props.values}
setValue={props.setValue}
setOpenMiningAuthForm={props.setOpenMiningAuthForm}
/>
)
case Settings.Security:
return <SecuritySettings />
case Settings.Reset:
Expand Down
1 change: 0 additions & 1 deletion gui-react/src/store/settings/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ export enum Settings {
Mining = 'mining',
BaseNode = 'baseNode',
Wallet = 'wallet',
Docker = 'docker',
// Logs = 'logs',
Security = 'security',
Reset = 'reset',
Expand Down

This file was deleted.

16 changes: 0 additions & 16 deletions ui/frontend/src/containers/SettingsContainer/SettingsDialog.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ import useAppStateStore from '../../store/appStateStore';
import ThemeSwitch from '../../components/ThemeSwitch';
import MergedMiningSettings from './MergedMiningSettings/MergedMiningSettings';
import BaseNodeSettings from './BaseNodeSettings/BaseNodeSettings';
import DockerSettings from './DockerSettings/DockerSettings';
import ShaMiningSettings from './ShaMiningSettings/ShaMiningSettings';
import ResetSettings from './ResetSettings/ResetSettings';
import { useTheme } from '@mui/material/styles';
Expand Down Expand Up @@ -135,15 +134,6 @@ function SettingsDialog() {
/>
),
},
{
label: 'Docker',
component: (
<DockerSettings
handleChange={handleFormChange}
formData={formData.dockerSettings}
/>
),
},
{
label: 'Reset',
component: <ResetSettings />,
Expand Down Expand Up @@ -232,12 +222,6 @@ function SettingsDialog() {
settings.mm_proxy.wallet_payment_address =
formData.shaMiningSettings.tariAddress;

// DOCKER SETTINGS
// dockerTag
settings.tag = formData.dockerSettings.dockerTag;
// dockerRegistry
settings.registry = formData.dockerSettings.dockerRegistry;

emit('tari://actions', {
Action: { type: 'SaveSettings', payload: settings },
});
Expand Down
6 changes: 0 additions & 6 deletions ui/frontend/src/containers/SettingsContainer/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,24 +19,18 @@ export const BaseNodeSettingsSchema = z.object({
runOnStartup: z.boolean(),
});

export const DockerSettingsSchema = z.object({
dockerTag: z.string(),
dockerRegistry: z.string(),
});

export const FormDataSchema = z.object({
mergedMiningSettings: MergedMiningSettingsSchema,
baseNodeSettings: BaseNodeSettingsSchema,
shaMiningSettings: ShaMiningSettingsSchema,
dockerSettings: DockerSettingsSchema,
});

export type ShaMiningSettingsType = z.infer<typeof ShaMiningSettingsSchema>;
export type MergedMiningSettingsType = z.infer<
typeof MergedMiningSettingsSchema
>;
export type BaseNodeSettingsType = z.infer<typeof BaseNodeSettingsSchema>;
export type DockerSettingsType = z.infer<typeof DockerSettingsSchema>;
export type FormDataType = z.infer<typeof FormDataSchema>;

export interface TabPanelProps {
Expand Down

0 comments on commit ac970ab

Please sign in to comment.