remove logging and left '@todo' comments to place events

decentdao · Jun 23, 2022 · 52d03fb · 52d03fb
1 parent 1d93640
commit 52d03fb
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 24 deletions.
diff --git a/src/Web3Provider.tsx b/src/Web3Provider.tsx
@@ -6,7 +6,6 @@ import { getWeb3modalOptions } from './helpers/web3ModalConfig';
 import { getLocalProvider, getFallbackProvider, getInjectedProvider } from './helpers';
 import { useProviderListeners } from './hooks/useProviderListeners';
 import { Web3ProviderContext } from './hooks/useWeb3Provider';
-import { logging } from './logging';
 
 const initialState: InitialState = {
   account: null,
@@ -105,7 +104,7 @@ export function Web3Provider({
   }, [connectDefaultProvider, config, web3Modal]);
 
   const disconnect: DisconnectFn = useCallback(() => {
-    logging('info', 'Wallet Action', 'Account Disconnected');
+    // @todo add 'account disconnected' event
     // switch to a default provider
     connectDefaultProvider();
   }, [connectDefaultProvider]);

diff --git a/src/hooks/useProviderListeners.ts b/src/hooks/useProviderListeners.ts
@@ -4,8 +4,6 @@ import { useState, useEffect } from 'react';
 import { ethers } from 'ethers';
 import Web3Modal from 'web3modal';
 
-import { logging } from '../logging';
-
 export const useProviderListeners = (
   web3Modal: Web3Modal,
   connectDefaultProvider: () => void,
@@ -19,16 +17,12 @@ export const useProviderListeners = (
     web3Modal.on('connect', _modalProvider => {
       // check that connected chain is supported
       if (!config.supportedChains.includes(parseInt(_modalProvider.chainId))) {
-        logging(
-          'error',
-          'Provider Error',
-          `Switch to a supported chain: ${config.supportedChains.join(', ')}`
-        );
+        // @todo add 'unsupported network' event
         // switch to a default provider
         connectDefaultProvider();
       } else {
         setModalProvider(_modalProvider);
-        logging('info', 'Wallet Action', `Account connected: ${_modalProvider.selectedAddress}`);
+        // @todo add 'connect' event
       }
     });
     return () => {
@@ -39,34 +33,29 @@ export const useProviderListeners = (
   useEffect(() => {
     const chainChangedCallback = (chainId: string) => {
       if (!config.supportedChains.includes(parseInt(chainId))) {
-        // check that connected chain is supported
-        logging(
-          'error',
-          'Provider Error',
-          `Switch to a supported chain: ${config.supportedChains.join(', ')}`
-        );
+        // @todo add 'unsupported network' event
         // switch to a default provider
         connectDefaultProvider();
       } else {
-        logging('info', 'Wallet Action', `Chain id updated: ${chainId}`);
+        // @todo add 'network changed' event
         connect();
       }
     };
 
     const accountsChangedCallback = (accounts: string[]) => {
       if (!accounts.length) {
-        logging('error', 'Wallet Action', 'Account access revoked');
+        // @todo add 'access revoked' event
         // switch to a default provider
         connectDefaultProvider();
         // remove listeners
         setModalProvider(null);
       } else {
-        logging('info', 'Wallet Action', 'Account changed');
+        // @todo add 'account changed event' event
         connect();
       }
     };
     const disconnectCallback = () => {
-      logging('error', 'Wallet Action', 'Account access revoked');
+      // @todo add 'access revoked' event
       // switch to a default provider
       connectDefaultProvider();
       // remove listeners

diff --git a/src/logging/index.ts b/src/logging/index.ts
@@ -1,4 +0,0 @@
-export function logging(type: string, actionType: string, message: string) {
-  console[type](actionType);
-  console[type](message);
-}