-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
JavaScript API does not work from within MetaMask's browser #4621
Comments
The exact same thing happens when connecting to another device using WalletConnect:
then MetaMask does absolutely nothing. |
What I described above happens about 95% of the time. Both in the case of using the in-app browser, and in the case of using another device connected throgh WalletConnect, if I retry several dozen times (e.g. clicking frenetically on the "send transaction" button in the example above, many many times), then about 5% of the times, the prompt to confirm the transaction will show up in MetaMask... ....But then, if I confirm the transaction, I get a stupid error saying "Internal JSON-RPC error". |
No response in 6 days to such a crippling issue?? You'd actually do a better service by not providing an in-app browser at all than providing such a broken one. At least you would't make users waste time trying. |
Hey @php4fan , thank you for report, sorry for taking so long to answer. Answering the issue of opening https://opentarget.org/test/stupid/mm/trans/ and pressing the send transaction, I'm not being able to reproduce it, everything went fine for me with an android device |
Can you try with an account that has no funds? I wonder if this is yet another manifestation of #4646, that is, the fact that when there are not enough funds for the transaction, MetaMask systematically fails to show an error message and just does nothing, immediately closes or just doesn't show up in the first place. |
This issue has been automatically marked as stale because it has not had recent activity in the last 90 days. It will be closed in 7 days. Thank you for your contributions. |
This is stupid. If no progress is made on an issue for a given amount of time, if anything that's a reason for giving the issue more attention, not for closing it. |
This issue has been automatically marked as stale because it has not had recent activity in the last 90 days. It will be closed in 7 days. Thank you for your contributions. |
Once more, please turn off this stupid thing |
This issue has been automatically marked as stale because it has not had recent activity in the last 90 days. It will be closed in 7 days. Thank you for your contributions. |
And again |
This issue has been automatically marked as stale because it has not had recent activity in the last 90 days. It will be closed in 7 days. Thank you for your contributions. |
When will you shut down this stupid bot? To close an issue because no progress has been made in fixing it is completely idiotic. |
This issue has been automatically marked as stale because it has not had recent activity in the last 90 days. It will be closed in 7 days. Thank you for your contributions. |
And again. BTW is this crap still being maintained? |
To Reproduce
Expected behavior
Should prompt me to accept or reject the transaction
Observed behavior
Does nothing.
If I haven't connected to the website before, it will show the prompt asking me to connect.
If the current network is not the BNB Smart Chain, it will prompt me to switch to that network.
But after that (or after none of that if that doesn't apply), it does not ask me to confirm the transaction.
Smartphone (please complete the following information):
to be added after bug submission by internal support / PM
Severity
The text was updated successfully, but these errors were encountered: