feat: add possibility to use async onSave method (#1079)

support of async onSave for custom hooks https://splunk.atlassian.net/browse/ADDON-62206
splunk · Feb 19, 2024 · 7dd6640 · 7dd6640
1 parent 5467ed9
commit 7dd6640
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/ui/src/components/BaseFormView.tsx b/ui/src/components/BaseFormView.tsx
@@ -522,7 +522,7 @@ class BaseFormView extends PureComponent<BaseFormProps, BaseFormState> {
         }
     };
 
-    handleSubmit = (event: React.MouseEvent | React.FormEvent) => {
+    handleSubmit = async (event: React.MouseEvent | React.FormEvent) => {
         event.preventDefault();
         this.clearErrorMsg();
         this.props.handleFormSubmit(/* isSubmitting */ true, /* closeEntity */ false);
@@ -536,7 +536,8 @@ class BaseFormView extends PureComponent<BaseFormProps, BaseFormState> {
         });
 
         if (this.hook && typeof this.hook.onSave === 'function') {
-            const validationPass = this.hook.onSave(this.datadict);
+            const validationPass = await this.hook.onSave(this.datadict);
+
             if (!validationPass) {
                 this.props.handleFormSubmit(/* isSubmitting */ false, /* closeEntity */ false);
                 return;
@@ -1251,6 +1252,7 @@ class BaseFormView extends PureComponent<BaseFormProps, BaseFormState> {
             }
             this.flag = false;
         }
+
         return (
             <div>
                 <form

diff --git a/ui/src/components/EntityModal/EntityModal.tsx b/ui/src/components/EntityModal/EntityModal.tsx
@@ -54,8 +54,8 @@ class EntityModal extends Component<EntityModalProps, EntityModalState> {
         this.props.handleRequestClose();
     };
 
-    handleSubmit: ButtonClickHandler = (e) => {
-        const result = this.form.current?.handleSubmit(e);
+    handleSubmit: ButtonClickHandler = async (e) => {
+        const result = await this.form.current?.handleSubmit(e);
         if (result) {
             this.handleRequestClose();
         }