Don't query batch on empty set of users (#143)

kscalelabs · Jun 17, 2024 · a058595 · a058595
1 parent f00e504
commit a058595
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/frontend/src/pages/Parts.tsx b/frontend/src/pages/Parts.tsx
@@ -31,7 +31,8 @@ const Parts = () => {
         partsQuery.forEach((part) => {
           ids.add(part.owner);
         });
-        setIdMap(await auth_api.getUserBatch(Array.from(ids)));
+        if (ids.size > 0)
+          setIdMap(await auth_api.getUserBatch(Array.from(ids)));
       } catch (err) {
         if (err instanceof Error) {
           setError(err.message);

diff --git a/frontend/src/pages/Robots.tsx b/frontend/src/pages/Robots.tsx
@@ -29,7 +29,8 @@ const Robots = () => {
         robotsQuery.forEach((robot) => {
           ids.add(robot.owner);
         });
-        setIdMap(await auth_api.getUserBatch(Array.from(ids)));
+        if (ids.size > 0)
+          setIdMap(await auth_api.getUserBatch(Array.from(ids)));
       } catch (err) {
         if (err instanceof Error) {
           addAlert(err.message, "error");