Fix Eslint errors

quickwit-oss · Jun 20, 2024 · eb3ffe5 · eb3ffe5
1 parent cb9c2c7
commit eb3ffe5
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 9 deletions.
diff --git a/quickwit/quickwit-ui/src/components/QueryEditor/AggregationEditor.tsx b/quickwit/quickwit-ui/src/components/QueryEditor/AggregationEditor.tsx
@@ -98,7 +98,6 @@ export function MetricKind(props: SearchComponentProps) {
 }
 
 export function AggregationKind(props: SearchComponentProps) {
-  props;
   const defaultAgg  = {
     histogram: {
       interval: "1d",

diff --git a/quickwit/quickwit-ui/src/components/SearchResult/AggregationResult.tsx b/quickwit/quickwit-ui/src/components/SearchResult/AggregationResult.tsx
@@ -47,7 +47,6 @@ export function AggregationResult({searchResponse}: {searchResponse: SearchRespo
           data: line.value,
        };
     });
-    series;
     // we don't customize colors because we would need a full palette.
     return (
     <LineChart

diff --git a/quickwit/quickwit-ui/src/components/SearchResult/SearchResult.tsx b/quickwit/quickwit-ui/src/components/SearchResult/SearchResult.tsx
@@ -56,13 +56,6 @@ interface SearchResultProps {
 }
 
 export default function SearchResult(props: SearchResultProps) {
-  if (props.queryRunning) {
-    return <Loader />;
-  }
-  if (props.searchError !== null) {
-    return ErrorResponseDisplay(props.searchError);
-  }
-
   const result = useMemo(() => {
     if (props.searchResponse == null || props.index == null) {
       return null;
@@ -73,6 +66,14 @@ export default function SearchResult(props: SearchResultProps) {
     }
   }, [props.searchResponse, props.index]);
 
+  if (props.queryRunning) {
+    return <Loader />;
+  }
+
+  if (props.searchError !== null) {
+    return ErrorResponseDisplay(props.searchError);
+  }
+
   if (props.searchResponse == null || props.index == null) {
     return <></>;
   }