Fixes the formik submit handler returning a promise for synchronous submit handlers which in turn immediately sets the isSubmitting to false.

[deepakjosp]

• Fixes "Save changes" button should be disabled in the ActionBlock component #2334

Description

• The async handleSubmit will always return a Promise, even if the submit handler inside doesn’t return one. This leads to unexpected bugs, as the Promise resolves immediately when using the mutate function of useMutation while the API call might still be in progress.
• In Formik, if a Promise is returned, it waits for it to resolve before setting isSubmitting to false. In our case, this happens immediately after the button click.
• Since we weren’t returning the result of the submit handler, it consistently returned a Promise once the handler executed, which caused this behavior.

Checklist

• I have made corresponding changes to the documentation.
• I have updated the types definition of modified exports.
• I have verified the functionality in some of the neeto web-apps.
• I have added tests that prove my fix is effective or that my feature works.
• I have added proper data-cy and data-testid attributes.
• I have added the necessary label (patch/minor/major - If package publish
  is required).

Reviewers

[deepakjosp]

@josephmathew900 _a please review.

[josephmathew900]

LGTM