temporarily disable execution for service queries with params (#1164)

finos · May 13, 2022 · f5ac673 · f5ac673
1 parent 1df0754
commit f5ac673
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/.changeset/happy-pumpkins-remember.md b/.changeset/happy-pumpkins-remember.md
@@ -0,0 +1,5 @@
+---
+"@finos/legend-studio": patch
+---
+
+Temporary disable executing service queries with parameters until workflow is supported.
diff --git a/...nd-studio/src/components/editor/edit-panel/service-editor/ServiceExecutionQueryEditor.tsx b/...nd-studio/src/components/editor/edit-panel/service-editor/ServiceExecutionQueryEditor.tsx
@@ -250,6 +250,10 @@ export const ServiceExecutionQueryEditor = observer(
     const importQuery = (): void => {
       queryState.setOpenQueryImporter(true);
     };
+    const temporaryDisableExecutionWithParams = Boolean(
+      Array.isArray(executionState.queryState.query.parameters) &&
+        executionState.queryState.query.parameters.length,
+    );
     // execute
     const execute = applicationStore.guardUnhandledError(() =>
       flowResult(executionState.execute()),
@@ -289,8 +293,16 @@ export const ServiceExecutionQueryEditor = observer(
             <button
               className="service-editor__execution__execute-btn"
               onClick={execute}
+              title={
+                temporaryDisableExecutionWithParams
+                  ? `Use query builder to execute queries with parameters`
+                  : `Execute`
+              }
               disabled={
-                executionState.isExecuting || executionState.isGeneratingPlan
+                executionState.isExecuting ||
+                executionState.isGeneratingPlan ||
+                // TODO: Enable once we allow setting parameters for execution
+                temporaryDisableExecutionWithParams
               }
               tabIndex={-1}
             >