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

Wallet Connect - Zapper.fi - Sign Request does not show the correct account to connect, transaction does not complete #16747

Open
virginiabalducci opened this issue Nov 11, 2024 · 3 comments
Assignees
Labels
bug Something isn't working wallet-team
Milestone

Comments

@virginiabalducci
Copy link

virginiabalducci commented Nov 11, 2024

Bug Report

Description

The Sign Request does not show the specific account that the dApp will connect to.
When scrolling through the information of the Sign Request, the app becomes laggy
There are many errors on the console

Steps to reproduce

  1. Go to https://yearn.fi/v3/
  2. Connect the dApp to the Desktop Wallet using a specific account, for the purpose of this testing 'Account 1' was connected
  3. Choose an asset to deposit for example USDC
  4. Choose an amount and select 'Deposit and Stake'
  5. On the Desktop app, when the Sign Request pops up, try to scroll. Notice the scroll is very laggy. Also notice that the Account to connect is not showing the correct information

Expected behavior

  • Sign Request to display the correct account to connect to

Actual behavior

app_20241111_145345.log

On the desktop side:

  • Console errors display
  • The account to connect to is not correctly displaying; it should show 'Account 1'
  • When scrolling, the pop-up becomes laggy

On the dApp side:

  • Transaction does not go through, after 5-10 minutes of the transaction loading, error shows 'Unknown RPC request'

The error seems to be related to the Fees, is there anything that can be done from the desktop app/s end? such as providing an error message (not enough balance for fees, etc) to guide the user on a possible reason why they cannot use this dApp?

ERR 2024-11-11 20:47:45.711Z qt error                                   topics="qt" tid=91663074 text="\nfromString (qrc:/StatusQ/Core/Utils/AmountsArithmetic.qml:87)\ngetEstimatedMaxFees (qrc:/app/AppLayouts/Wallet/services/dapps/plugins/SignRequestPlugin.qml:418)\ngetEstimatedFeesStatus (qrc:/app/AppLayouts/Wallet/services/dapps/plugins/SignRequestPlugin.qml:469)\nupdateFeesParamsToPassedObj (qrc:/app/AppLayouts/Wallet/services/dapps/plugins/SignRequestPlugin.qml:289)\nonTriggered (qrc:/app/AppLayouts/Wallet/services/dapps/plugins/SignRequestPlugin.qml:665)" file=qrc:/StatusQ/Core/Utils/AmountsArithmetic.qml:87 category=default
ERR 2024-11-11 20:47:45.711Z qt error                                   topics="qt" tid=91663074 text="\nfromString (qrc:/StatusQ/Core/Utils/AmountsArithmetic.qml:87)\ngetEstimatedMaxFees (qrc:/app/AppLayouts/Wallet/services/dapps/plugins/SignRequestPlugin.qml:419)\ngetEstimatedFeesStatus (qrc:/app/AppLayouts/Wallet/services/dapps/plugins/SignRequestPlugin.qml:469)\nupdateFeesParamsToPassedObj (qrc:/app/AppLayouts/Wallet/services/dapps/plugins/SignRequestPlugin.qml:289)\nonTriggered (qrc:/app/AppLayouts/Wallet/services/dapps/plugins/SignRequestPlugin.qml:665)" file=qrc:/StatusQ/Core/Utils/AmountsArithmetic.qml:87 category=default
ERR 2024-11-11 20:47:45.712Z qt error                                   topics="qt" tid=91663074 text="\nfromString (qrc:/StatusQ/Core/Utils/AmountsArithmetic.qml:87)\ngetEstimatedMaxFees (qrc:/app/AppLayouts/Wallet/services/dapps/plugins/SignRequestPlugin.qml:420)\ngetEstimatedFeesStatus (qrc:/app/AppLayouts/Wallet/services/dapps/plugins/SignRequestPlugin.qml:469)\nupdateFeesParamsToPassedObj (qrc:/app/AppLayouts/Wallet/services/dapps/plugins/SignRequestPlugin.qml:289)\nonTriggered (qrc:/app/AppLayouts/Wallet/services/dapps/plugins/SignRequestPlugin.qml:665)" file=qrc:/StatusQ/Core/Utils/AmountsArithmetic.qml:87 category=default

Image

Image

Screen.Recording.2024-11-11.at.5.46.59.PM.mov

Test with USDT
There seems to be an error as well, and while dApp connects, the transaction doesn't go through.

Image

Screen.Recording.2024-11-11.at.6.01.31.PM.mov

Console log:
app_20241111_145345.log

Additional Information

@virginiabalducci virginiabalducci added bug Something isn't working wallet-team labels Nov 11, 2024
@virginiabalducci virginiabalducci added this to the 2.32.0 Beta milestone Nov 11, 2024
@virginiabalducci
Copy link
Author

cc @alexjba for visibility

@alexjba
Copy link
Contributor

alexjba commented Nov 12, 2024

There are several issues here:

  1. Incorrect value. This is what the dApp sends \"value\":\"0x0\"
  2. Laggy UX. Looks like it's due to the periodic fees. All WC actions are done on the main thread and the fee request is blocking for a short period of time. Now we do periodic checks for the fees, and it blocks every few seconds.
  3. Incorrect account in the modal - Something we need to investigate. The address is case sensitive, but the matching should be case insensitive.

@alexjba
Copy link
Contributor

alexjba commented Nov 12, 2024

Note: The dApp behaves differently based on the Browser. On Safari it won't trigger the transaction request, on Chrome it triggers the tx request, but the value is 0

@alexjba alexjba self-assigned this Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working wallet-team
Projects
Status: In Progress
2 participants