Leo Vigna

Results 15 comments of Leo Vigna

I also get the same issue. I downgraded to 0.14.4 as well.

Is there any update on the progress of this issue? Multi-CSI support is critical for our Harvester use case as some of our workloads won't work optimally with Longhorn (IOPS)....

Ok thanks for the info. Is there an estimated timeline for that release (eg. 1mo, 6mo, 1yr)? Would just help us out for planning our migration to Harvester. Thx!

Should I add a changeset. Please let me know if this PR is welcome.

I have had the same error following the official ProtocolKit tutorial and debugged the root issue by manually deploying the Safe. Issue comes from here: ```typescript // src/adapters/ethers/contracts/SafeProxyFactoryEthersContract.ts async createProxy({...

Note while this patch (see #512) fixes the decoding, the core underlying issue is the contract object stored by the SDK does not seem to match the deployed contract's interface.

Decoding the return value also works :+1: However, it seems your solution @franvf will only work with web3.js since ethers does not support `getPastEvents` out of the box.

Thanks. I would also recommend the actual interface used by the SDK (as mentioned in comment above) as this could cause other issues if parts of the code rely on...

Was using v1.2.0. I see v1.3.0 was recently released, was the issue fixed there?

It seems this file was not changed https://github.com/safe-global/safe-core-sdk/blob/main/packages/protocol-kit/src/adapters/ethers/contracts/SafeProxyFactory/SafeProxyFactoryEthersContract.ts as can also be seen by my PR https://github.com/safe-global/safe-core-sdk/pull/512/files I would assume the error persists due to the above described issue with...