From 5956d482e8f8f5979aaf372ccce793bc196a4e17 Mon Sep 17 00:00:00 2001
From: Julius Volz <julius.volz@gmail.com>
Date: Mon, 9 Sep 2024 11:19:47 +0200
Subject: [PATCH] Remove a few unneeded comments and component props

Signed-off-by: Julius Volz <julius.volz@gmail.com>
---
 web/ui/mantine-ui/src/pages/query/ExpressionInput.tsx   | 2 +-
 web/ui/mantine-ui/src/pages/query/QueryPanel.tsx        | 1 -
 web/ui/mantine-ui/src/pages/query/TreeView.tsx          | 2 --
 web/ui/mantine-ui/src/pages/targets/ScrapePoolsList.tsx | 1 -
 4 files changed, 1 insertion(+), 5 deletions(-)

diff --git a/web/ui/mantine-ui/src/pages/query/ExpressionInput.tsx b/web/ui/mantine-ui/src/pages/query/ExpressionInput.tsx
index d7dbf0dd69..dce05c1814 100644
--- a/web/ui/mantine-ui/src/pages/query/ExpressionInput.tsx
+++ b/web/ui/mantine-ui/src/pages/query/ExpressionInput.tsx
@@ -206,7 +206,7 @@ const ExpressionInput: FC<ExpressionInputProps> = ({
     enableLinter,
     enableQueryHistory,
     queryHistory,
-  ]); // TODO: Make this depend on external settings changes, maybe use dynamic config compartment again.
+  ]); // TODO: Maybe use dynamic config compartment again as in the old UI?
 
   return (
     <Group align="flex-start" wrap="nowrap" gap="xs">
diff --git a/web/ui/mantine-ui/src/pages/query/QueryPanel.tsx b/web/ui/mantine-ui/src/pages/query/QueryPanel.tsx
index a172311828..24eb8c59cb 100644
--- a/web/ui/mantine-ui/src/pages/query/QueryPanel.tsx
+++ b/web/ui/mantine-ui/src/pages/query/QueryPanel.tsx
@@ -123,7 +123,6 @@ const QueryPanel: FC<PanelProps> = ({ idx, metricNames }) => {
           >
             <TreeView
               panelIdx={idx}
-              retriggerIdx={retriggerIdx}
               selectedNode={selectedNode}
               setSelectedNode={setSelectedNode}
               closeTreeView={() => {
diff --git a/web/ui/mantine-ui/src/pages/query/TreeView.tsx b/web/ui/mantine-ui/src/pages/query/TreeView.tsx
index 1455e43774..3a631f1e32 100644
--- a/web/ui/mantine-ui/src/pages/query/TreeView.tsx
+++ b/web/ui/mantine-ui/src/pages/query/TreeView.tsx
@@ -7,8 +7,6 @@ import { Card, CloseButton } from "@mantine/core";
 
 const TreeView: FC<{
   panelIdx: number;
-  // TODO: Do we need retriggerIdx for the tree view AST parsing? Maybe for children!
-  retriggerIdx: number;
   selectedNode: {
     id: string;
     node: ASTNode;
diff --git a/web/ui/mantine-ui/src/pages/targets/ScrapePoolsList.tsx b/web/ui/mantine-ui/src/pages/targets/ScrapePoolsList.tsx
index 290543fc5b..8790f45221 100644
--- a/web/ui/mantine-ui/src/pages/targets/ScrapePoolsList.tsx
+++ b/web/ui/mantine-ui/src/pages/targets/ScrapePoolsList.tsx
@@ -326,7 +326,6 @@ const ScrapePoolList: FC<ScrapePoolListProp> = ({
                                 }}
                               >
                                 <Table.Td valign="top">
-                                  {/* TODO: Process target URL like in old UI */}
                                   <EndpointLink
                                     endpoint={target.scrapeUrl}
                                     globalUrl={target.globalUrl}