checkConnection()

[0xZakk]

nextjs-gitcoin-passport/app/page.tsx

Lines 33 to 46 in e44fd53

 checkConnection() 
 async function checkConnection() { 
   try { 
     const provider = new ethers.providers.Web3Provider(window.ethereum) 
     const accounts = await provider.listAccounts() 
     if (accounts && accounts[0]) { 
       setConnected(true) 
       setAddress(accounts[0]) 
       checkPassport(accounts[0]) 
     } 
   } catch (err) { 
     console.log('not connected...') 
   } 
 } 

Do we need to invoke and then define the function on the next line like this?

[Nader Dabit]

hey, I'm not sure I follow the question here

this function just checks their connection and automatically fetches their score if they've already connected.

[0xZakk]

My question is just about the formatting (defining and then invoking a function, all inside of a function). Could we make the parent function async and move everything from checkConnection out into the parent function? Or is there a technical reason it has to be inside a separate function?

[Nader Dabit]

AH! yes, great question, you cannot (or are not supposed to do this) due to how useEffect works, there's some discussion about those details here

[0xZakk]

Ahh okay - good to know