Fix: Fixed UI errors and warnings (open-metadata#23837)

* Fix: Fixed UI errors and warnings

* Fix: Removed commented code

* Fix: removed the post interface

* Fix: reverted few changes

* Fix: Addressed PR comments

* Fix: removed unnecessary types

* Fix: removed unnecessary types

* Fix: fixed unit test issue

Author: Rohit0301

openmetadata-ui/src/main/resources/ui/src/components/ActivityFeed/ActivityFeedCardNew/CommentCard.component.tsx

@@ -15,7 +15,7 @@ import classNames from 'classnames';
 import { compare } from 'fast-json-patch';
 import { useCallback, useEffect, useMemo, useRef, useState } from 'react';
 import { Link } from 'react-router-dom';
-import { Thread } from '../../../generated/entity/feed/thread';
+import { Post, Thread } from '../../../generated/entity/feed/thread';
 import { useUserProfile } from '../../../hooks/user-profile/useUserProfile';
 import {
   formatDateTime,
@@ -37,7 +37,7 @@ import ActivityFeedActions from '../Shared/ActivityFeedActions';
 
 interface CommentCardInterface {
   feed: Thread;
-  post: any;
+  post: Post;
   isLastReply: boolean;
   closeFeedEditor: () => void;
 }

openmetadata-ui/src/main/resources/ui/src/components/Domain/DomainDetailPage/DomainDetailPage.component.tsx

@@ -87,7 +87,7 @@ const DomainDetailPage = () => {
     }
   };
 
-  const handleDomainDelete = (id: string) => {
+  const handleDomainDelete = () => {
     // Navigate back to domains listing page after deletion
     navigate(ROUTES.DOMAIN);
   };

openmetadata-ui/src/main/resources/ui/src/components/Glossary/GlossaryStatusBadge/GlossaryStatusBadge.component.tsx

@@ -19,7 +19,10 @@ export const GlossaryStatusBadge = ({ status }: { status: EntityStatus }) => {
   return (
     <Space>
       <Divider className="m-x-xs h-6" type="vertical" />
-      <StatusBadge label={status} status={StatusClass[status]} />
+      <StatusBadge
+        label={status}
+        status={StatusClass[status as keyof typeof StatusClass]}
+      />
     </Space>
   );
 };

openmetadata-ui/src/main/resources/ui/src/components/Glossary/GlossaryTermTab/GlossaryTermTab.component.tsx

@@ -774,7 +774,7 @@ const GlossaryTermTab = ({ isGlossary, className }: GlossaryTermTabProps) => {
                 <StatusBadge
                   dataTestId={termFQN + '-status'}
                   label={status}
-                  status={StatusClass[status]}
+                  status={StatusClass[status as keyof typeof StatusClass]}
                 />
               </div>
             </Popover>

openmetadata-ui/src/main/resources/ui/src/components/LineageTable/LineageTable.tsx

@@ -629,7 +629,7 @@ const LineageTable: FC<{ entity: SourceType }> = ({ entity }) => {
             ? 'fromEntity'
             : 'toEntity',
         sorter: true,
-        render: (record?: SearchSourceAlias) => (
+        render: (record?: SearchSourceAlias & { type: EntityType }) => (
           <Link
             to={getEntityLinkFromType(
               record?.fullyQualifiedName ?? '',
@@ -671,7 +671,7 @@ const LineageTable: FC<{ entity: SourceType }> = ({ entity }) => {
             ? 'fromEntity'
             : 'toEntity',
         sorter: true,
-        render: (record?: SearchSourceAlias) => (
+        render: (record?: SearchSourceAlias & { type?: EntityType }) => (
           <Link
             to={getEntityLinkFromType(
               record?.fullyQualifiedName ?? '',

openmetadata-ui/src/main/resources/ui/src/components/MyData/CustomizableComponents/CustomiseHomeModal/CustomiseHomeModal.test.tsx

@@ -49,7 +49,7 @@ jest.mock('antd', () => {
 
   return {
     ...antd,
-    Modal: ({ children, open, onCancel, ...props }: any) =>
+    Modal: ({ children, open, ...props }: any) =>
       open ? (
         <div data-testid="modal" {...props}>
           {children}

openmetadata-ui/src/main/resources/ui/src/components/Settings/Applications/AppRunsHistory/AppRunsHistory.test.tsx

@@ -114,7 +114,7 @@ jest.mock('../../../../utils/date-time/DateTimeUtils', () => ({
   getEpochMillisForPastDays: jest.fn().mockReturnValue('startDay'),
   getIntervalInMilliseconds: jest.fn().mockReturnValue('interval'),
   formatDuration: jest.fn().mockReturnValue('formatDuration'),
-  formatDurationToHHMMSS: jest.fn().mockImplementation((_ms) => {
+  formatDurationToHHMMSS: jest.fn().mockImplementation(() => {
     // Return a consistent formatted duration for all cases
     return '02:30:15';
   }),

openmetadata-ui/src/main/resources/ui/src/components/Settings/Services/Ingestion/IngestionRecentRun/IngestionRecentRun.test.tsx

@@ -18,6 +18,7 @@ import {
   render,
   screen,
 } from '@testing-library/react';
+import { Status } from '../../../../../generated/entity/applications/appRunRecord';
 import { IngestionPipeline } from '../../../../../generated/entity/services/ingestionPipelines/ingestionPipeline';
 import { EXECUTION_RUNS, FAILURE } from '../../../../../mocks/Ingestion.mock';
 import { mockDataInsightApplicationRun } from '../../../../../mocks/LogsViewerPage.mock';
@@ -438,23 +439,23 @@ describe('Test IngestionRecentRun component', () => {
       const unorderedAppRuns = [
         {
           runId: 'app-run-2',
-          status: 'Success',
+          status: Status.Success,
           startTime: 1667307000,
           timestamp: 1667307000,
           endTime: 1667307003,
           appId: 'app-2',
         },
         {
           runId: 'app-run-1',
-          status: 'Failed',
+          status: Status.Failed,
           startTime: 1667301000,
           timestamp: 1667301000,
           endTime: 1667301003,
           appId: 'app-1',
         },
         {
           runId: 'app-run-3',
-          status: 'Running',
+          status: Status.Running,
           startTime: 1667309000,
           timestamp: 1667309000,
           endTime: 1667309003,

openmetadata-ui/src/main/resources/ui/src/components/common/MUIAsyncTreeSelect/MUIAsyncTreeSelect.tsx

@@ -499,22 +499,25 @@ const MUIAsyncTreeSelect: FC<MUIAsyncTreeSelectProps> = ({
   );
 
   // Handle input blur to restore selected value if no new selection
-  const handleInputBlur = useCallback(
-    (e: React.FocusEvent) => {
-      // Restore selected value in single-select if no new selection
-      if (!multiple && selectedOptions.length > 0) {
-        const selectedLabel = selectedOptions[0].label;
-        if (inputValue !== selectedLabel) {
-          setInputValue(selectedLabel);
-          debouncedSetSearchTerm(''); // Clear search to show all options
-        }
+  const handleInputBlur = useCallback(() => {
+    // Restore selected value in single-select if no new selection
+    if (!multiple && selectedOptions.length > 0) {
+      const selectedLabel = selectedOptions[0].label;
+      if (inputValue !== selectedLabel) {
+        setInputValue(selectedLabel);
+        debouncedSetSearchTerm(''); // Clear search to show all options
       }
+    }
 
-      // Call existing blur handler for dropdown management
-      handleBlur(e);
-    },
-    [multiple, selectedOptions, inputValue, handleBlur, debouncedSetSearchTerm]
-  );
+    // Call existing blur handler for dropdown management
+    handleBlur();
+  }, [
+    multiple,
+    selectedOptions,
+    inputValue,
+    handleBlur,
+    debouncedSetSearchTerm,
+  ]);
 
   // Calculate if has clearable value
   const hasClearableValue = multiple

openmetadata-ui/src/main/resources/ui/src/components/common/MUIAsyncTreeSelect/atoms/TreeContent/TreeContent.tsx

@@ -23,13 +23,13 @@ export interface TreeContentProps {
   error: string | null;
   hasData: boolean;
   children: ReactNode;
-  selectedItems?: string[] | null;
+  selectedItems?: string | null;
   expandedItems?: string[];
   focusedItem?: string;
   apiRef?: MutableRefObject<any>;
   loadingMessage?: string;
   noDataMessage?: string;
-  onNodeToggle?: SimpleTreeViewProps['onExpandedItemsChange'];
+  onNodeToggle?: SimpleTreeViewProps<boolean>['onExpandedItemsChange'];
   onFocusedItemChange?: (event: React.SyntheticEvent, itemId: string) => void;
   onItemClick?: (event: React.MouseEvent, itemId: string) => void;
 }