Cavemanon/piratepoet
7
Fork 0
Code Pull Requests Actions Releases 16 Activity

Fix: Only start polling the server on success response

Browse Source 
MAILPOET-4300
This commit is contained in:
Oluwaseun Olorunsola
2022-07-06 13:12:15 +01:00
committed by Veljko V
parent ef5e45ba32
commit 1b103c653f
1 changed files with 13 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
mailpoet/assets/js/src/common/authorize_sender_email_modal.tsx
View File

@ -122,6 +122,8 @@ function AuthorizeSenderEmailModal({
return null;
}
clearCurrentInterval(setIntervalId.current);
makeApiRequest(senderEmailAddress)
.then((res) => {
const response = Boolean(res?.data);
@ -130,6 +132,16 @@ function AuthorizeSenderEmailModal({
if (response) {
// if pending or already authorized perform the check ahead
executeAction();
// start polling on success response
setIntervalStopTime.current = moment()
.add(STOP_POLLING_AFTER, 'hours')
.valueOf();
setIntervalId.current = setInterval(
executeAction,
1000 * SET_INTERVAL_SECONDS,
);
}
})
.catch(() => {
@ -137,15 +149,7 @@ function AuthorizeSenderEmailModal({
setShowLoader(false);
});
clearCurrentInterval(setIntervalId.current);
setIntervalStopTime.current = moment()
.add(STOP_POLLING_AFTER, 'hours')
.valueOf();
const invervalID = setInterval(executeAction, 1000 * SET_INTERVAL_SECONDS);
setIntervalId.current = invervalID;
return () => clearCurrentInterval(invervalID);
return () => clearCurrentInterval(setIntervalId.current);
}, [senderEmailAddress]); // eslint-disable-line react-hooks/exhaustive-deps
return (