remove todos

Author: imaitland

app/routes/devices.$id.hardware.$hardware_name.history.tsx

@@ -45,7 +45,6 @@ export async function loader({ params, request }: LoaderFunctionArgs) {
 
   const properties = searchParams.get("properties")?.split(",");
 
-  // TODO History api is in flux.
   const {
     data: { data },
   } = await Evolver.history({

app/routes/devices.$id.hardware.tsx

@@ -25,7 +25,6 @@ export const handle = {
 export default function Hardware() {
   const { id, hardware_name } = useParams();
   const [queryParams] = useSearchParams();
-  // TODO: figure this out, should submit to nearest layout route with a loader
   const loaderData = useRouteLoaderData<typeof loader>("routes/devices.$id");
   let evolverConfig = {} as EvolverConfigWithoutDefaults;
 

app/utils/pingDevice.server.ts

@@ -28,19 +28,16 @@ export async function pingDevice(
     const evolverDescription = await Evolver.describe({
       client: evolverClient,
     });
-    // TODO: Confirm this is all accessible
     const {
       config: { name },
-      id,
     } = evolverDescription.data;
-
-    return { online: true, name, id };
+    return { online: true, name };
   } catch (error) {
     if (error instanceof DOMException && error.name === "AbortError") {
       console.info(
         `request to device ${evolver_url_addr} timed out after ${timeout}ms, now designated offline`,
       );
     }
-    return { online: false, name: "unknown", id: "unknown" };
+    return { online: false, name: "unknown" };
   }
 }