fix: re-render while autoload

[gasolin]

fix the rerender issue while autoload === true

[PaulRBerg]

Hi @gasolin, thanks for your contribution.

Could you add more details to the problem targeted by this change? When does the UI re-render? A GIF may be helpful.

[gasolin]

@PaulRBerg sorry for the inconvinient.

During test project development I found when autoload set to true(the default case),
the related effect turns to an infinite loop (can monitor by add console.log in that useEffect)

It might be related to the web3Modal.cachedProvider keep changing

[PaulRBerg]

I understand, but I can't reproduce this. Could you maybe upload a GIF that shows how the re-render occurs on your computer?

[gasolin]

UI doesn't change, but your CPU/FAN will roar when you open the page longer

[PaulRBerg]

My CPU doesn't roar, but I have a high-end spec so maybe that's the reason.

@TomAFrench can you reproduce this?

Sorry to be a pain in the neck @gasolin, but I would not introduce more complexity in the template if the change is a no-op.

[Neozaru]

Could definitely reproduce this as this caused me exceed my Metamask's Infura quota :D

Env :

• Windows 10
• Google Chrome
• Metamask (Kovan with default params)

Steps to reproduce :

• Add the line console.log('ETH2 launch successful') somewhere in the base component (ex: App() function).
• Open dApp with dev tools open, and check console.

@gasolin Thanks. That solved the issue for me.

Come back to me for questions / if you can't reproduce.

[PaulRBerg]

Whoah, you are right. Just added the console log as you suggested @Neozaru and this is what I'm seeing:

Enregistrement.de.l.ecran.2020-12-20.a.14.17.04.mov

Crazy. Looking at @gasolin's fix now.