feat: Update AppStatusModal to conditionally show timeline based on deployment app type and remove outdated comment in processDeploymentStatusDetailsData

Author: AbhishekA1509

src/Shared/Components/AppStatusModal/AppStatusModal.component.tsx

@@ -79,7 +79,9 @@ const AppStatusModal = ({
                             ? appDetails.installedAppId
                             : appDetails.appId,
                     envId: appDetails.environmentId,
-                    showTimeline: selectedTab === AppStatusModalTabType.DEPLOYMENT_STATUS,
+                    showTimeline:
+                        selectedTab === AppStatusModalTabType.DEPLOYMENT_STATUS &&
+                        appDetails.deploymentAppType !== DeploymentAppTypes.HELM,
                     virtualEnvironmentConfig: appDetails.isVirtualEnvironment
                         ? {
                               processVirtualEnvironmentDeploymentData,

src/Shared/Components/DeploymentStatusBreakdown/utils.tsx

@@ -244,7 +244,6 @@ export const processDeploymentStatusDetailsData = (
 
     // After initial processing will mark all unavailable timelines [present before last invalid state] as success
     PHYSICAL_ENV_DEPLOYMENT_TIMELINE_ORDER.forEach((timelineStatusType, index) => {
-        // TODO: Confirm for findRight approach is fine? since, for e.g, i believe apply start should be before apply synced
         const element = findRight(data.timelines, getPredicate(timelineStatusType))
         const timelineData = deploymentData.deploymentStatusBreakdown[timelineStatusType]