Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: unable to determine contract std shows infinite spinning wheel #14712

Closed
worldlyjohn opened this issue May 16, 2022 · 7 comments
Closed
Labels
area-transactions Sev2-normal Normal severity; minor loss of service or inconvenience. team-confirmations-secure-ux DEPRECATED: please use "team-confirmations" label instead type-bug

Comments

@worldlyjohn
Copy link
Contributor

worldlyjohn commented May 16, 2022

Describe the bug

During a dapp contractInteraction (token approval), I get this error.
I'm unclear if it has to do with the contract itself or sentry, but it looks like we aren't handling the sentry error properly (perhaps we should try/catch and continue rather than halt the app like this?

image

Steps to reproduce

1.Have MKR in your wallet (0x9f8f72aa9304c8b593d555f12ef6589cc3a579a2)
2. Go to Zerion's Maker page
3. Attempt to sell some MKR for ETH
4. Zerion button turns to "Approve"
5. Click "Approve"

I see the spinning wheel and it never shows the Txn Approval Screen

Error messages or log output

image

Version

10.14.3

Build type

No response

Browser

Chrome

Operating system

MacOS

Hardware wallet

No response

Additional context

No response

@bschorchit
Copy link

Related to: #14187

@bschorchit bschorchit added Sev2-normal Normal severity; minor loss of service or inconvenience. area-transactions team-confirmations-secure-ux DEPRECATED: please use "team-confirmations" label instead labels May 16, 2022
@worldlyjohn worldlyjohn changed the title [Bug]: Cannot handle Sentry HTTP 429 status code [Bug]: unable to determine contract std shows infinite spinning wheel May 16, 2022
@NinjaTurtle007
Copy link

Here are more people having it with some somewhat useful reports
https://community.metamask.io/t/i-cant-send-just-the-mkr-token/19753/16

@bschorchit
Copy link

Closing this as the fix has been merged. This should be fixed for users in one of our next releases.

@bschorchit
Copy link

Reopening this as we still need to create a new controller version and update the controller version being used on extension.

@bschorchit bschorchit reopened this May 26, 2022
@bschorchit
Copy link

This will be done with: #14891

@NinjaTurtle007
Copy link

So when we can expect release of a version that allows you to send MKR token?

@bschorchit
Copy link

This will be fixed in extension release v10.19 @NinjaTurtle007
Closing this here as we're tracking it already here: #14187

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-transactions Sev2-normal Normal severity; minor loss of service or inconvenience. team-confirmations-secure-ux DEPRECATED: please use "team-confirmations" label instead type-bug
Projects
None yet
Development

No branches or pull requests

3 participants