You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Sep 28, 2022. It is now read-only.
I don't believe selectedAddress isn't a standardized property on window.ethereum. While it may exist in MetaMask, there is no guarantee that it exists on any other wallet. Recommend having a fallback to the result of wallet_requestAccounts or eth_coinbase or eth_accounts. Currently, if the connect wallet doesn't have a selectedAddress property on window.ethereum the console gets spammed with Uncaught (in promise) TypeError: Cannot read property 'toLowerCase' of undefined because this function always returns undefined.
The text was updated successfully, but these errors were encountered:
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
dai.js/packages/dai/src/eth/AccountsService.js
Lines 205 to 209 in 29a594e
I don't believe
selectedAddress
isn't a standardized property onwindow.ethereum
. While it may exist in MetaMask, there is no guarantee that it exists on any other wallet. Recommend having a fallback to the result ofwallet_requestAccounts
oreth_coinbase
oreth_accounts
. Currently, if the connect wallet doesn't have aselectedAddress
property onwindow.ethereum
the console gets spammed withUncaught (in promise) TypeError: Cannot read property 'toLowerCase' of undefined
because this function always returnsundefined
.The text was updated successfully, but these errors were encountered: