metamask-mobile icon indicating copy to clipboard operation
metamask-mobile copied to clipboard

WalletConnect Reject action callback issue : ERC20 contracts

Open nareshkatta99 opened this issue 3 years ago • 4 comments

Describe the bug When we use WalletConnect and initiate a ERC20 approve operation and we 'Reject' the transaction, it is not reporting back about rejection to WalletConnect

The same 'Reject' operation on any other contracts is working properly.

To Reproduce Steps to reproduce the behavior

  1. Go to enzyme.finance, select Polygon Mainnet and use WalletConnect to connect.
  2. Scan the QR Code with Metamask mobile app
  3. Scroll down to 'Popular vaults' and select any one of them (ex Insula™ Phaeacia).
  4. Click on 'Deposit', then select 'Dai' in the popup.
  5. Enter some number less than your Dai Balance and then click on 'Approve'.
  6. In Metamask Android app, Click Reject.
  7. Even after 5-10 mins, the WalletConnect will not detect that the User rejected the transaction.

Expected behavior Just like for other contract operations, say Create a Vault in this case, which does not involve ERC20 transaction, the Metamask mobile app must report the rejection to WalletConnect.

Smartphone (please complete the following information):

  • Device: RedMi Note 8 pro
  • OS: Android 11
  • App Version 5.1.1

to be added after bug submission by internal support / PM Severity

  • How critical is the impact of this bug on a user?
  • Add stats if available on % of customers impacted
  • Is this visible to all users?
  • Is this tech debt?

nareshkatta99 avatar Jun 01 '22 12:06 nareshkatta99

@tommasini any update on this?

nareshkatta99 avatar Jun 17 '22 08:06 nareshkatta99

Hey @nareshkatta99, we are trying to solve it as soon as we can, thanks again for reporting!

tommasini avatar Jun 17 '22 11:06 tommasini

@tommasini Please help us know when can we expect an update on this.

nareshkatta99 avatar Aug 08 '22 19:08 nareshkatta99

Hi @nareshkatta99, thank you for your patience! We don't have any relevant updates at the moment

gantunesr avatar Aug 09 '22 13:08 gantunesr

This issue was closed because it has been stalled for 7 days with no activity. If you feel this was closed in error please reopen and provide evidence on the current production app. Thank you for your contributions.

github-actions[bot] avatar Jun 16 '23 23:06 github-actions[bot]