[Fix] Hotfix for handling responses from n-API (#139)

Fixes #138 This is a temporary fix until we get rid of the non-federated deployment option in general
neurobagel · Apr 19, 2024 · c6b0ad7 · c6b0ad7
1 parent ac1840d
commit c6b0ad7
Showing 1 changed file with 24 additions and 14 deletions.
diff --git a/src/App.tsx b/src/App.tsx
@@ -281,21 +281,31 @@ function App() {
     const url: string = constructQueryURL();
     try {
       const response = await axios.get(url);
-      setResult(response.data);
-      switch (response.data.nodes_response_status) {
-        case 'partial success': {
-          response.data.errors.forEach((error: NodeError) => {
-            enqueueSnackbar(`${error.node_name} failed to respond`, { variant: 'warning' });
-          });
-          break;
-        }
-        case 'fail': {
-          enqueueSnackbar('Error: All nodes failed to respond', { variant: 'error' });
-          break;
-        }
-        default: {
-          break;
+      // TODO: remove this branch once there is no more non-federation option
+      if (isFederationAPI) {
+        setResult(response.data);
+        switch (response.data.nodes_response_status) {
+          case 'partial success': {
+            response.data.errors.forEach((error: NodeError) => {
+              enqueueSnackbar(`${error.node_name} failed to respond`, { variant: 'warning' });
+            });
+            break;
+          }
+          case 'fail': {
+            enqueueSnackbar('Error: All nodes failed to respond', { variant: 'error' });
+            break;
+          }
+          default: {
+            break;
+          }
         }
+      } else {
+        const myResponse = {
+          responses: response.data,
+          nodes_response_status: 'success',
+          errors: [],
+        };
+        setResult(myResponse);
       }
     } catch (error) {
       enqueueSnackbar('Failed to retrieve results', { variant: 'error' });