contract-proxy-kit icon indicating copy to clipboard operation
contract-proxy-kit copied to clipboard

Wrong request. Error: The data couldn’t be read because it isn’t in the correct format when using Safe via walletconnect

Open mmv08 opened this issue 5 years ago • 6 comments
trafficstars

The error is thrown when trying to call execTransactions with Safe (walletconnect):

here is the example app where you can reproduce this: https://github.com/gnosis/cpk-compound-example/blob/master/src/components/CompoundForm.tsx#L70

mmv08 avatar Jan 27 '20 14:01 mmv08

The error in the issue title was thrown when using safe rinkeby on iOS

on android the following error is thrown: Error: Could not handle custom call: java.lang.ClassCastException: java.lang.Double cannot be cast to java.lang.String

mmv08 avatar Jan 28 '20 16:01 mmv08

In general it seems not to work with walletconnect: https://rinkeby.etherscan.io/tx/0xbd2ab441819d441ec43eca138ec5588935e5dafa01c53a4585f0b54b90c33ead

with metamask mobile the tx reverts, but it works with the same wallet but in an extension

mmv08 avatar Jan 28 '20 16:01 mmv08

For the linked transaction you run out of gas, up the gas limit

rmeissner avatar Feb 20 '20 10:02 rmeissner

@cag issue is that operation for the multi send needs to be a string. Could this please be changed.

rmeissner avatar Feb 20 '20 10:02 rmeissner

The issue still persists. You should be able to reproduce it on master branch in https://github.com/gnosis/cpk-compound-tutorial/

mmv08 avatar Mar 10 '20 15:03 mmv08

This should be fixed as soon https://github.com/gnosis/safe-android/issues/540 is implemented, but we'll have to revert "pass operation as string" changes

mmv08 avatar Mar 11 '20 15:03 mmv08