Merge pull request #2926 from devtron-labs/feat/deployment-metrics-charts

feat: replace recharts with Chart component in deployment metrics

Author: Elessar1802

.eslintignore

@@ -92,7 +92,6 @@ src/components/app/details/cdDetails/CDDetails.tsx
 src/components/app/details/cicdHistory/History.components.tsx
 src/components/app/details/main.tsx
 src/components/app/details/metrics/BenchmarkModal.tsx
-src/components/app/details/metrics/DeploymentMetrics.tsx
 src/components/app/details/metrics/DeploymentTable.tsx
 src/components/app/details/metrics/DeploymentTableModal.tsx
 src/components/app/details/metrics/deploymentMetrics.service.ts

package.json

@@ -4,7 +4,7 @@
     "private": true,
     "homepage": "/dashboard",
     "dependencies": {
-        "@devtron-labs/devtron-fe-common-lib": "1.20.3-pre-9",
+        "@devtron-labs/devtron-fe-common-lib": "1.20.3-alpha-9",
         "@esbuild-plugins/node-globals-polyfill": "0.2.3",
         "@rjsf/core": "^5.13.3",
         "@rjsf/utils": "^5.13.3",
@@ -31,7 +31,6 @@
         "react-router-dom": "^5.3.4",
         "react-select": "5.8.0",
         "react-virtualized": "^9.22.5",
-        "recharts": "^2.1.9",
         "rollup-plugin-node-polyfills": "0.2.1",
         "rxjs": "^7.5.4",
         "sockjs-client": "1.6.1",
@@ -79,7 +78,6 @@
         "@types/react-dom": "17.0.13",
         "@types/react-router-dom": "^5.3.3",
         "@types/react-transition-group": "^4.4.4",
-        "@types/recharts": "^1.8.23",
         "@types/recompose": "^0.30.10",
         "@typescript-eslint/eslint-plugin": "8.3.0",
         "@typescript-eslint/parser": "8.3.0",

src/components/app/details/main.tsx

@@ -432,20 +432,9 @@ export default function AppDetailsPage() {
                                 <CIDetails key={appId} filteredEnvIds={_filteredEnvIds} />
                             </AIAgentContextSetterWrapper>
                         </Route>
-                        <Route
-                            path={`${path}/${URLS.APP_DEPLOYMENT_METRICS}/:envId(\\d+)?`}
-                            render={(props) => {
-                                const envId = props.match.params.envId
-                                const match = {
-                                    ...props.match,
-                                    params: {
-                                        appId: appId,
-                                        envId: envId,
-                                    },
-                                }
-                                return <DeploymentMetrics {...props} match={match} filteredEnvIds={_filteredEnvIds} />
-                            }}
-                        />
+                        <Route path={`${path}/${URLS.APP_DEPLOYMENT_METRICS}/:envId(\\d+)?`}>
+                            <DeploymentMetrics filteredEnvIds={_filteredEnvIds} />
+                        </Route>
                         <Route
                             path={`${path}/${URLS.APP_CD_DETAILS}/:envId(\\d+)?/:pipelineId(\\d+)?/:triggerId(\\d+)?`}
                         >