DEV Community

Discussion on: Which functions/methods do you...

View post

Replies for: Nice snippet. The new Promise callback should not be an async function. By passing an async fn to new Promise you’re “double promising”. Refactori...

 
joelbonetr profile image
JoelBonetR 🥇 • • Edited

Having it as async lets you handle the response, the error and any action to perform either it went OK or KO, so it can be customised specifically on it's environment.

myPoll.then( res => res ).catch( err => console.error( 'myCustomError', err )).finally( () => setExecutionFinished(true));
Enter fullscreen mode Exit fullscreen mode

instead handling it as generic inside the polling function.

But sure you can tweak it as you wish to fullfill your needs 😄