Ethers.js: provider.getSigner() is not function

[billtlee]

Hi,

For some reason I am getting the error provider.getSigner() is not a function with the following code. Can someone tell me what I am doing wrong?

  import ethers from 'ethers';

  if (typeof window.ethereum !== 'undefined' || (typeof window.web3 !== 'undefined')) {

    // Web3 browser user detected. You can now use the provider.
    const provider = window['ethereum'] || window.web3.currentProvider

    const accounts = await window.ethereum.enable();

    console.log('accounts: ', accounts);
    console.log('provider: ', provider);

    const signer = provider.getSigner();
}

[ricmoo]

That provider is an EIP-1193 provider, not an ethers.js provider. You need to wrap it in a new ethers.providers.Web3Provider(window.ethereum) to get an ethers.js provider object.

See: https://docs.ethers.io/ethers.js/html/cookbook-providers.html#metamask

That example in the docs is using the legacy window.web3, but it should work fine with a window.ethereum provider too. :)

[billtlee]

Thanks Richard. I got it to work.

import { ethers } from 'ethers';
  if (typeof window.ethereum !== 'undefined' || (typeof window.web3 !== 'undefined')) {

    // Web3 browser user detected. You can now use the provider.
    const accounts = await window.ethereum.enable();
    // const curProvider = window['ethereum'] || window.web3.currentProvider

    const provider = new ethers.providers.Web3Provider(window.ethereum);

    console.log('accounts: ', accounts);
    console.log('provider: ', provider);

    const signer = provider.getSigner();

[ricmoo]

Awesome, closing this now, but feel free to re-open, or continue commenting (I monitor closed issues).

Thanks! :)

[yash-deore]

Hi,
I am getting the error I have mentioned below for somw reason.Please help.

Unhandled Rejection (TypeError): provider.getSigner is not a constructor

if (window.ethereum !== 'undefined') {
await this.requestAccount();
const provider = new ethers.providers.Web3Provider(window.ethereum);
const signer = new provider.getSigner();
const contract = new ethers.Contract(greeterAddress, Greeter.abi, signer);
const transaction = await contract.setGreeting('this.state.greeting');
await transaction.wait();
}

[ricmoo]

Do not use new here:

// ------------v
const signer = new provider.getSigner();

[yash-deore]

Thanks a lot my code worked.
Appreciate your anticipation.

On Wed, 28 Apr 2021, 02:22 Richard Moore, @.*> wrote:

Do not use new here:

// ------------v
const signer = new provider.getSigner();

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ethers-io/ethers.js/issues/457#issuecomment-827922178,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS2QZZGIEADIYTAX3AWRPRDTK4PYTANCNFSM4G7NIZYA
.