From bb04a8f135089a8c3bfecd10b700563d5fa20762 Mon Sep 17 00:00:00 2001
From: Arsen Alkamyan <44676488+arsengit@users.noreply.github.com>
Date: Tue, 17 May 2022 20:41:54 +0400
Subject: [PATCH] [fix] Wrap Overview tab cards with ErrorBoundary (#1757)

---
 .../RunOverviewTab/RunOverviewTab.tsx         | 80 +++++++++++--------
 1 file changed, 48 insertions(+), 32 deletions(-)

diff --git a/aim/web/ui/src/pages/RunDetail/RunOverviewTab/RunOverviewTab.tsx b/aim/web/ui/src/pages/RunDetail/RunOverviewTab/RunOverviewTab.tsx
index efc3867557..ad004bdf0f 100644
--- a/aim/web/ui/src/pages/RunDetail/RunOverviewTab/RunOverviewTab.tsx
+++ b/aim/web/ui/src/pages/RunDetail/RunOverviewTab/RunOverviewTab.tsx
@@ -72,55 +72,71 @@ function RunOverviewTab({ runData, runHash }: IRunOverviewTabProps) {
           ) : (
             <>
               {_.isEmpty(cardsData?.runParams) ? null : (
-                <RunOverviewTabParamsCard
-                  runParams={cardsData?.runParams}
-                  isRunInfoLoading={runData?.isRunInfoLoading}
-                />
+                <ErrorBoundary>
+                  <RunOverviewTabParamsCard
+                    runParams={cardsData?.runParams}
+                    isRunInfoLoading={runData?.isRunInfoLoading}
+                  />
+                </ErrorBoundary>
               )}
               {_.isEmpty(cardsData?.runMetricsBatch) ? null : (
-                <RunOverviewTabMetricsCard
-                  isLoading={runData?.isRunBatchLoading}
-                  type='metric'
-                  runBatch={cardsData?.runMetricsBatch}
-                />
+                <ErrorBoundary>
+                  <RunOverviewTabMetricsCard
+                    isLoading={runData?.isRunBatchLoading}
+                    type='metric'
+                    runBatch={cardsData?.runMetricsBatch}
+                  />
+                </ErrorBoundary>
               )}
 
               {_.isEmpty(cardsData?.runSystemBatch) ? null : (
-                <RunOverviewTabMetricsCard
-                  isLoading={runData?.isRunBatchLoading}
-                  type='systemMetric'
-                  runBatch={cardsData?.runSystemBatch}
-                />
+                <ErrorBoundary>
+                  <RunOverviewTabMetricsCard
+                    isLoading={runData?.isRunBatchLoading}
+                    type='systemMetric'
+                    runBatch={cardsData?.runSystemBatch}
+                  />
+                </ErrorBoundary>
               )}
               {_.isEmpty(cardsData.cliArguments) ? null : (
-                <RunOverviewTabCLIArgumentsCard
-                  cliArguments={cardsData.cliArguments}
-                  isRunInfoLoading={runData?.isRunInfoLoading}
-                />
+                <ErrorBoundary>
+                  <RunOverviewTabCLIArgumentsCard
+                    cliArguments={cardsData.cliArguments}
+                    isRunInfoLoading={runData?.isRunInfoLoading}
+                  />
+                </ErrorBoundary>
               )}
               {_.isEmpty(cardsData.envVariables) ? null : (
-                <RunOverviewTabEnvVariablesCard
-                  envVariables={cardsData.envVariables}
-                  isRunInfoLoading={runData?.isRunInfoLoading}
-                />
+                <ErrorBoundary>
+                  <RunOverviewTabEnvVariablesCard
+                    envVariables={cardsData.envVariables}
+                    isRunInfoLoading={runData?.isRunInfoLoading}
+                  />
+                </ErrorBoundary>
               )}
               {_.isEmpty(cardsData.packages) ? null : (
-                <RunOverviewTabPackagesCard
-                  packages={cardsData.packages}
-                  isRunInfoLoading={runData?.isRunInfoLoading}
-                />
+                <ErrorBoundary>
+                  <RunOverviewTabPackagesCard
+                    packages={cardsData.packages}
+                    isRunInfoLoading={runData?.isRunInfoLoading}
+                  />
+                </ErrorBoundary>
               )}
               {_.isEmpty(cardsData.gitInfo) ? null : (
-                <GitInfoCard data={cardsData.gitInfo} />
+                <ErrorBoundary>
+                  <GitInfoCard data={cardsData.gitInfo} />
+                </ErrorBoundary>
               )}
             </>
           )}
         </div>
-        <RunOverviewSidebar
-          runHash={runHash}
-          info={runData.runInfo}
-          traces={runData.runTraces}
-        />
+        <ErrorBoundary>
+          <RunOverviewSidebar
+            runHash={runHash}
+            info={runData.runInfo}
+            traces={runData.runTraces}
+          />
+        </ErrorBoundary>
       </section>
     </ErrorBoundary>
   );