Merge branch 'develop' into chore/revamp-manifest-component-header

Author: RohitRaj011

.eslintignore

@@ -5,7 +5,7 @@
 .eslintrc.js
 vite.config.mts
 
-# The following files have eslint errors/warningssrc/App.tsx
+# The following files have eslint errors/warnings
 src/App.tsx
 src/Pages/GlobalConfigurations/Authorization/APITokens/__tests__/ApiTokens.test.tsx
 src/components/AppSelector/AppSelector.tsx
@@ -82,9 +82,7 @@ src/components/ClusterNodes/constants.ts
 src/components/Hotjar/Hotjar.tsx
 src/components/Jobs/ExpandedRow/ExpandedRow.tsx
 src/components/Jobs/JobDetails/JobDetails.tsx
-src/components/Jobs/JobList/JobListContainer.tsx
 src/components/Jobs/JobList/JobListView.tsx
-src/components/Jobs/JobList/JobsList.tsx
 src/components/Jobs/Jobs.tsx
 src/components/Jobs/JobsEmptyState.tsx
 src/components/Jobs/Service.ts
@@ -159,16 +157,12 @@ src/components/app/details/triggerView/workflow/nodes/triggerCINode.tsx
 src/components/app/details/triggerView/workflow/nodes/triggerPrePostCDNode.tsx
 src/components/app/details/triggerView/workflow/nodes/workflow.utils.tsx
 src/components/app/grepSSEworker.ts
-src/components/app/list-new/AppList.tsx
 src/components/app/list-new/AppListService.ts
 src/components/app/list-new/GenericAppList.tsx
 src/components/app/list-new/HelmAppList.tsx
-src/components/app/list/AppListView.tsx
-src/components/app/list/DevtronAppListContainer.tsx
 src/components/app/list/TriggerUrl.tsx
 src/components/app/list/__tests__/AppListView.test.tsx
 src/components/app/list/appList.modal.ts
-src/components/app/list/appStatus/AppStatus.tsx
 src/components/app/list/emptyView/Empty.tsx
 src/components/app/list/expandedRow/ExpandedRow.tsx
 src/components/app/service.ts
@@ -286,7 +280,6 @@ src/components/common/eaEmptyState/EAEmptyState.tsx
 src/components/common/edge/rectangularEdge.tsx
 src/components/common/edge/straightEdge.tsx
 src/components/common/errorBoundary.tsx
-src/components/common/filter/filters.tsx
 src/components/common/formFields/CopyButton.tsx
 src/components/common/formFields/CustomPassword.tsx
 src/components/common/formFields/DevtronSwitch.tsx
@@ -417,7 +410,6 @@ src/components/v2/appDetails/appDetails.store.ts
 src/components/v2/appDetails/appDetails.type.ts
 src/components/v2/appDetails/ea/EAAppDetail.component.tsx
 src/components/v2/appDetails/k8Resource/FilterResource.tsx
-src/components/v2/appDetails/k8Resource/K8Resource.component.tsx
 src/components/v2/appDetails/k8Resource/NodeTreeTabList.tsx
 src/components/v2/appDetails/k8Resource/nodeDetail/EphemeralContainerDrawer.tsx
 src/components/v2/appDetails/k8Resource/nodeDetail/NodeDetail.component.tsx
@@ -440,7 +432,6 @@ src/components/v2/appDetails/k8Resource/nodeDetail/nodeDetail.util.ts
 src/components/v2/appDetails/k8Resource/nodeType/Node.component.tsx
 src/components/v2/appDetails/k8Resource/nodeType/NodeDelete.component.tsx
 src/components/v2/appDetails/k8Resource/nodeType/NodeTree.component.tsx
-src/components/v2/appDetails/k8Resource/nodeType/PodHeader.component.tsx
 src/components/v2/appDetails/k8Resource/nodeType/PodTabSection.tsx
 src/components/v2/appDetails/k8Resource/nodeType/__tests__/NodeTree.component.test.ts
 src/components/v2/appDetails/k8Resource/nodeType/nodeType.util.ts

package.json

@@ -4,7 +4,7 @@
     "private": true,
     "homepage": "/dashboard",
     "dependencies": {
-        "@devtron-labs/devtron-fe-common-lib": "0.3.6",
+        "@devtron-labs/devtron-fe-common-lib": "0.3.7",
         "@esbuild-plugins/node-globals-polyfill": "0.2.3",
         "@rjsf/core": "^5.13.3",
         "@rjsf/utils": "^5.13.3",

src/App.tsx

@@ -98,6 +98,8 @@ export default function App() {
             }
         } else {
             didMountRef.current = true
+            // Removing any toast explicitly due to race condition of offline toast for some users
+            ToastManager.dismissToast(onlineToastRef.current)
         }
     }, [isOnline])
 

src/assets/gif/ic-empty-select-cluster.gif

@@ -24,13 +24,15 @@ import {
     PipelineType,
     CommonNodeAttr,
     WorkflowType,
+    getUrlWithSearchParams,
 } from '@devtron-labs/devtron-fe-common-lib'
 import { CdPipelineResult, CiPipelineResult, WorkflowResult, CiPipeline } from '../app/details/triggerView/types'
 import { WebhookListResponse } from '../ciPipeline/Webhook/types'
 import { processWorkflow } from '../app/details/triggerView/workflow.service'
 import { WorkflowTrigger } from '../app/details/triggerView/config'
 import { ModuleNameMap, Routes, URLS } from '../../config'
 import {
+    AppGroupFilterConfig,
     AppGroupList,
     CIConfigListType,
     CheckPermissionResponse,
@@ -41,6 +43,7 @@ import {
     EnvDeploymentStatusType,
     EnvGroupListResponse,
     EnvGroupResponse,
+    GetEnvAppListParamsType,
     WorkflowsResponseType,
 } from './AppGroup.types'
 import { getModuleConfigured } from '../app/details/appDetails/appDetails.service'
@@ -190,26 +193,20 @@ export const getConfigAppList = (envId: number, appIds: string): Promise<ConfigA
 }
 
 export const getEnvAppList = (
-    params?: {
-        envName?: string
-        clusterIds?: string
-        offset?: string
-        size?: string
-    },
+    filterConfig?: Partial<AppGroupFilterConfig>,
     signal?: AbortSignal,
 ): Promise<EnvAppType> => {
     const options = signal ? { signal } : null
-
-    if (params) {
-        const urlParams = Object.entries(params).map(([key, value]) => {
-            if (!value) {
-                return
-            }
-            return `${key}=${value}`
-        })
-        return get(`${Routes.ENVIRONMENT_APPS}?${urlParams.filter((s) => s).join('&')}`, options)
+    const { searchKey = '', cluster = [], offset = 0, pageSize = 20 } = filterConfig ?? {}
+    const params: GetEnvAppListParamsType = {
+        envName: searchKey,
+        clusterIds: cluster?.join(),
+        offset,
+        size: pageSize,
     }
-    return get(Routes.ENVIRONMENT_APPS, options)
+
+    const url = getUrlWithSearchParams(Routes.ENVIRONMENT_APPS, params)
+    return get(url, options)
 }
 
 export const getDeploymentStatus = (envId: number, appIds: string): Promise<EnvDeploymentStatusType> => {

src/assets/icons/ic-select-cluster.svg

@@ -27,6 +27,7 @@ import {
     AppInfoListType,
     GVKType,
     RuntimeParamsListItemType,
+    UseUrlFiltersReturnType,
 } from '@devtron-labs/devtron-fe-common-lib'
 import { MultiValue } from 'react-select'
 import { WebhookPayloads } from '../app/details/triggerView/types'
@@ -254,9 +255,21 @@ export interface ApplicationRouteType {
     fetchEnvConfig: () => void
 }
 
-export interface EnvironmentsListViewType {
-    removeAllFilters: () => void
+export interface AppGroupFilterConfig
+    extends Pick<UseUrlFiltersReturnType<never>, 'searchKey' | 'offset' | 'pageSize'> {
+    cluster: string[]
+}
+
+export interface GetEnvAppListParamsType extends Pick<AppGroupFilterConfig, 'offset'> {
+    size: number
+    envName: string
+    clusterIds: string
+}
+
+export interface EnvironmentsListViewType
+    extends Pick<UseUrlFiltersReturnType<never>, 'changePage' | 'changePageSize' | 'clearFilters'> {
     isSuperAdmin: boolean
+    filterConfig: AppGroupFilterConfig
 }
 
 export interface EnvironmentLinkProps {
@@ -591,3 +604,9 @@ export interface ManageAppsResponseType {
     id: string
     error: string
 }
+
+export enum AppGroupUrlFilters {
+    cluster = 'cluster',
+}
+
+export interface AppGroupUrlFiltersType extends Record<AppGroupUrlFilters, string[]> {}