Move clear interval until after things are setup

MutinyWallet / mutiny-web

The frontend for Mutiny Wallet

https://app.mutinywallet.com

MIT License

174 stars 49 forks source link

Move clear interval until after things are setup #1187

Closed benthecarman closed 2 weeks ago

benthecarman commented 2 weeks ago

This fixed one version of setup getting stuck for me

cloudflare-pages[bot] commented 2 weeks ago

Deploying mutiny-web with Cloudflare Pages

Latest commit:	`7a75c90`
Status:	✅ Deploy successful!
Preview URL:	https://977a46af.mutiny-web.pages.dev
Branch Preview URL:	https://mv-interval.mutiny-web.pages.dev

View logs

futurepaul commented 2 weeks ago

My worry is that we end up with more scenarios where the interval doesn't get cleared... really wish we had a better way of doing this timeout thing it's really not been fun.

benthecarman commented 2 weeks ago

Made it so we'll clear it in the catch block as well, should be guaranteed to clear now