Add an http check on form updates as they default to success when the…

…re is an error, ie 403
NitorCreations · Mar 24, 2024 · 7d7f80c · 7d7f80c
1 parent df23d8e
commit 7d7f80c
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/nflow-explorer/src/workflow-instance/manage/UpdateWorkflowInstanceSignalForm.tsx b/nflow-explorer/src/workflow-instance/manage/UpdateWorkflowInstanceSignalForm.tsx
@@ -25,7 +25,10 @@ const UpdateWorkflowInstanceSignalForm = function (props: {
       signal: values.signal,
       reason: values.reason
     }).then((reason: any) => {
-      if (reason.error) {
+      if (!reason.ok) {
+        setErrorMsg(`HTTP error! status: ${reason.status}`);
+        setConfirmationMsg('');
+      } else if (reason.error) {
         setErrorMsg(reason.error);
         setConfirmationMsg('');
       } else {

diff --git a/nflow-explorer/src/workflow-instance/manage/UpdateWorkflowInstanceStateForm.tsx b/nflow-explorer/src/workflow-instance/manage/UpdateWorkflowInstanceStateForm.tsx
@@ -26,7 +26,10 @@ const UpdateWorkflowInstanceStateForm = function (props: {
       ),
       actionDescription: values.actionDescription
     }).then((reason: any) => {
-      if (reason.error) {
+      if (!reason.ok) {
+        setErrorMsg(`HTTP error! status: ${reason.status}`);
+        setConfirmationMsg('');
+      } else if (reason.error) {
         setErrorMsg(reason.error);
         setConfirmationMsg('');
       } else {

diff --git a/nflow-explorer/src/workflow-instance/manage/UpdateWorkflowInstanceStateVariableForm.tsx b/nflow-explorer/src/workflow-instance/manage/UpdateWorkflowInstanceStateVariableForm.tsx
@@ -23,7 +23,10 @@ const UpdateWorkflowInstanceStateVariableForm = function (props: {
       },
       actionDescription: values.actionDescription
     }).then((reason: any) => {
-      if (reason.error) {
+      if (!reason.ok) {
+        setErrorMsg(`HTTP error! status: ${reason.status}`);
+        setConfirmationMsg('');
+      } else if (reason.error) {
         setErrorMsg(reason.error);
         setConfirmationMsg('');
       } else {