feat: drop issues section from log viewer

- issues section remove from the context of builds and tests details only Close #993
kernelci · Feb 26, 2025 · 5326832 · 5326832
1 parent fbc783e
commit 5326832
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 6 deletions.
diff --git a/dashboard/src/components/BuildDetails/BuildDetails.tsx b/dashboard/src/components/BuildDetails/BuildDetails.tsx
@@ -272,6 +272,7 @@ const BuildDetails = ({
             <LogOrJsonSheetContent
               type={sheetType}
               jsonContent={jsonContent}
+              hideIssueSection
               logUrl={data?.log_url}
               logExcerpt={data?.log_excerpt}
             />

diff --git a/dashboard/src/components/Sheet/LogOrJsonSheetContent.tsx b/dashboard/src/components/Sheet/LogOrJsonSheetContent.tsx
@@ -26,6 +26,7 @@ interface ILogSheet {
   navigationLogsActions?: TNavigationLogActions;
   currentLinkProps?: LinkProps;
   issues?: TIssue[];
+  hideIssueSection?: boolean;
   status?: UseQueryResult['status'];
   error?: UseQueryResult['error'];
 }
@@ -43,6 +44,7 @@ export const LogOrJsonSheetContent = ({
   navigationLogsActions,
   currentLinkProps,
   issues,
+  hideIssueSection,
   status,
   error,
 }: ILogSheet): JSX.Element => {
@@ -66,12 +68,15 @@ export const LogOrJsonSheetContent = ({
             logExcerpt={logExcerpt}
             isLoading={navigationLogsActions?.isLoading}
           />
-          <IssueSection
-            data={issues}
-            status={status ?? 'success'}
-            error={error?.message}
-            variant="warning"
-          />
+
+          {!hideIssueSection && (
+            <IssueSection
+              data={issues}
+              status={status ?? 'success'}
+              error={error?.message}
+              variant="warning"
+            />
+          )}
         </>
       ) : (
         <ReactJsonView

diff --git a/dashboard/src/components/TestDetails/TestDetails.tsx b/dashboard/src/components/TestDetails/TestDetails.tsx
@@ -364,6 +364,7 @@ const TestDetails = ({ breadcrumb }: TestsDetailsProps): JSX.Element => {
             type={sheetType}
             jsonContent={jsonContent}
             logUrl={data?.log_url}
+            hideIssueSection
             logExcerpt={data?.log_excerpt}
           />
         </Sheet>