review fix

Lifted the agents count and consequent search bar disabling to index.tsx
inrupt · May 13, 2024 · 5b5ab70 · 5b5ab70
1 parent f33d507
commit 5b5ab70
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/pages/index.tsx b/pages/index.tsx
@@ -61,7 +61,11 @@ export default function ManagePage() {
               })
             }
             searchString={value}
-            onNotEmptyAgents={() => setDisableSearch(false)}
+            onAgentUpdate={(agentCount) => {
+              agentCount === 0
+                ? setDisableSearch(true)
+                : setDisableSearch(false);
+            }}
           />
         </div>
       </main>

diff --git a/src/components/ListAgents/ListAgentsWithVCs.tsx b/src/components/ListAgents/ListAgentsWithVCs.tsx
@@ -83,11 +83,11 @@ function SuspendedList({
 export default function ListAgentsWithVCs({
   onDetails,
   searchString,
-  onNotEmptyAgents,
+  onAgentUpdate,
 }: {
   onDetails: (_agent: string) => void;
   searchString: string;
-  onNotEmptyAgents?: () => void;
+  onAgentUpdate?: (agentCount: number) => void;
 }) {
   const { session, accessEndpoint, endpointConfiguration } =
     useContext(SessionContext);
@@ -113,10 +113,12 @@ export default function ListAgentsWithVCs({
   }, [accessEndpoint, endpointConfiguration, session, isValidAccessGrant]);
 
   useEffect(() => {
-    if (agents !== undefined && agents.length > 0) {
-      if (onNotEmptyAgents) {
-        onNotEmptyAgents();
+    if (typeof onAgentUpdate === "function") {
+      let agentsCount = 0;
+      if (agents !== undefined) {
+        agentsCount = agents.length;
       }
+      onAgentUpdate(agentsCount);
     }
   }, [agents]);