Update ContractStore.walletConnected in AccountListener when the user disconnects the wallet (#1119)

* check if response is null before checking response.ok

* Update ContractStore in AccountListener when the user disconnects the wallet

Author: wonjun-opensource

contracts/test/vault/rebase.js

@@ -51,7 +51,7 @@ describe("Vault rebase pausing", async () => {
     );
   });
 
-  it("Should allow governor tonpause rebasing", async () => {
+  it("Should allow governor to pause rebasing", async () => {
     let { vault, governor } = await loadFixture(defaultFixture);
     await vault.connect(governor).pauseRebase();
   });

dapp/src/components/AccountListener.js

@@ -90,6 +90,9 @@ const AccountListener = (props) => {
         s.allowances = {}
         s.balances = {}
       })
+      ContractStore.update((s) => {
+        s.walletConnected = false
+      })
       PoolStore.update((s) => {
         s.claimable_ogn = null
         s.lp_tokens = null

dapp/src/components/buySell/BalanceHeader.js

@@ -243,7 +243,9 @@ const BalanceHeader = ({
             <Statistic
               title={fbt('Balance', 'OUSD Balance')}
               value={
-                !isNaN(parseFloat(displayedBalance)) && ousdBalanceLoaded
+                walletConnected &&
+                !isNaN(parseFloat(displayedBalance)) &&
+                ousdBalanceLoaded
                   ? displayedBalance
                   : '--.--'
               }

dapp/src/components/wrap/BalanceHeaderWrapped.js

@@ -257,7 +257,9 @@ const BalanceHeaderWrapped = ({
             <Statistic
               title={fbt('wOUSD Balance', 'wOUSD Balance')}
               value={
-                !isNaN(parseFloat(displayedWousdBalance)) && wousdBalanceLoaded
+                walletConnected &&
+                !isNaN(parseFloat(displayedWousdBalance)) &&
+                wousdBalanceLoaded
                   ? displayedWousdBalance
                   : '--.--'
               }

dapp/src/services/apy.service.js

@@ -12,7 +12,7 @@ export default class ApyService {
           throw new Error(`Unexpected days param: ${days}`)
         }
         const response = await fetch(endpoint)
-        if (!response.ok) {
+        if (!response || !response.ok) {
           throw new Error(`Failed to fetch ${days} day APY`, err)
         }
         const json = await response.json()

dapp/src/services/balances.service.js

@@ -44,7 +44,7 @@ export default class BalancesService {
       body: JSON.stringify(data),
     })
 
-    if (!response.ok) {
+    if (!response || !response.ok) {
       throw new Error(
         `Could not fetch balances from Alchemy http status: ${response.status}`
       )

dapp/src/services/transaction-history-page.service.js

@@ -19,7 +19,7 @@ export default class TransactionHistoryPageService {
       }/${account.toLowerCase()}/history?per_page=${transactionHistoryItemsPerPage}&page=${page}${filter_param}`
     )
 
-    if (!response.ok) {
+    if (!response || !response.ok) {
       throw new Error('Failed fetching history from analytics')
     }
 

dapp/src/services/transaction-history.service.js

@@ -13,7 +13,7 @@ export default class TransactionHistoryService {
         this.baseURL
       }/${account.toLowerCase()}/history?per_page=1000000${filter_param}`
     )
-    if (!response.ok) {
+    if (!response || !response.ok) {
       throw new Error('Failed fetching history from analytics')
     }
 

dapp/src/utils/contracts.js

@@ -389,6 +389,9 @@ export async function setupContracts(account, library, chainId, fetchId) {
 
   const fetchCreditsPerToken = async () => {
     try {
+      if (!walletConnected) {
+        return
+      }
       const response = await fetch(process.env.CREDITS_ANALYTICS_ENDPOINT)
       if (response.ok) {
         const json = await response.json()