frontend: common: Remove Logviewer Terminal from common

Because they are not used in many modules, and so they should not be loaded all the time by putting them in the common/index. Additionally, this means that the code is only loaded when the pod details route is loaded. Signed-off-by: René Dudfield <[email protected]>
headlamp-k8s · May 7, 2024 · d781b47 · d781b47
1 parent b7570fd
commit d781b47
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/frontend/src/components/common/index.test.ts b/frontend/src/components/common/index.test.ts
@@ -11,6 +11,8 @@ const avoidCheck = [
  'ActionsNotifier',
  'AlertNotification',
  'ErrorBoundary',
+ 'LogViewer',
+ 'Terminal',
 ];
 
 const checkExports = [
@@ -27,7 +29,6 @@ const checkExports = [
  'LabelListItem',
  'Link',
  'Loader',
- 'LogViewer',
  'NamespacesAutocomplete',
  'NameValueTable',
  'Resource',
@@ -37,7 +38,6 @@ const checkExports = [
  'ShowHideLabel',
  'SimpleTable',
  'Tabs',
- 'Terminal',
  'TileChart',
  'TimezoneSelect',
  'Tooltip',

diff --git a/frontend/src/components/common/index.ts b/frontend/src/components/common/index.ts
@@ -14,7 +14,6 @@ export * from './Link';
 export { default as Link } from './Link';
 export * from './Loader';
 export { default as Loader } from './Loader';
-export * from './LogViewer';
 export * from './NameValueTable';
 export { default as NameValueTable } from './NameValueTable';
 export * from './ObjectEventList';
@@ -34,8 +33,6 @@ export * from './SimpleTable';
 export { default as SimpleTable } from './SimpleTable';
 export * from './Tabs';
 export { default as Tabs } from './Tabs';
-export * from './Terminal';
-export { default as Terminal } from './Terminal';
 export * from './TileChart';
 export { default as TileChart } from './TileChart';
 export * from './TimezoneSelect';