diff --git a/app/client/src/sagas/SuperUserSagas.tsx b/app/client/src/sagas/SuperUserSagas.tsx index 1cc67b6869..f633b22208 100644 --- a/app/client/src/sagas/SuperUserSagas.tsx +++ b/app/client/src/sagas/SuperUserSagas.tsx @@ -110,34 +110,32 @@ function* SendTestEmail(action: ReduxAction) { try { const response = yield call(UserApi.sendTestEmail, action.payload); const currentUser = yield select(getCurrentUser); - let actionElement; - if (response.data) { - actionElement = ( - <> -
- window.open(EMAIL_SETUP_DOC, "blank")}> - {createMessage(TEST_EMAIL_SUCCESS_TROUBLESHOOT)} - - - ); + const isValidResponse = yield validateResponse(response); + + if (isValidResponse) { + let actionElement; + if (response.data) { + actionElement = ( + <> +
+ window.open(EMAIL_SETUP_DOC, "blank")}> + {createMessage(TEST_EMAIL_SUCCESS_TROUBLESHOOT)} + + + ); + } + Toaster.show({ + actionElement, + text: createMessage( + response.data + ? TEST_EMAIL_SUCCESS(currentUser?.email) + : TEST_EMAIL_FAILURE, + ), + hideProgressBar: true, + variant: response.data ? Variant.info : Variant.danger, + }); } - Toaster.show({ - actionElement, - text: createMessage( - response.data - ? TEST_EMAIL_SUCCESS(currentUser?.email) - : TEST_EMAIL_FAILURE, - ), - hideProgressBar: true, - variant: response.data ? Variant.info : Variant.danger, - }); - } catch (e) { - Toaster.show({ - text: e?.message || createMessage(TEST_EMAIL_FAILURE), - hideProgressBar: true, - variant: Variant.danger, - }); - } + } catch (e) {} } function* InitSuperUserSaga(action: ReduxAction) {