New UI for `approve` transactions

Greetings!

I do believe this is bug;

I have next code in typescirpt:

 if (allowance.lt(payAmount)) {
      await erc20Contract
        .approve(contract.address, payAmount)
        .then((tx) => handle(tx))
    }

In the new mm UI, user has to explicitly set up approve value or user can click default and accept value from dApp. In my case value of payAmount. For example:

If payAmount is 42*10**18 : that is 42 ERC token with decimals 18 to approve. So if user clicks default it expected value in new form is 42, but actual is 42*10**18

1 Like

Hi! Would you be able to report this on Github?

1 Like

Sure, I have just created new github issue.

I can not add links here, so here is gh reference:

MetaMask/metamask-extension/issues/19019

This is no bug. Sorry for inconvenience;
closed gh issue.

2 Likes

This topic was automatically closed 30 days after the last reply. New replies are no longer allowed.