NethermindEth
Change FeeEstimation type to use u64 and u128 instead of felt.Felt
Spec ref: https://github.com/starkware-libs/starknet-specs/blob/master/api/starknet_api_openrpc.json#L3478
Depends on: https://github.com/NethermindEth/starknet.go/issues/709
@thiagodeev thanks! can you provide me any extra context to complete the task, is my first time contributing to this project
You can take a look at the examples folder to learn how to work with the library, especially the invoke example; it shows how to estimate the fee. The goal is to change the FeeEstimation type returned by the EstimateFee method, make it use u64 and u128 values when necessary (this is the spec link describing which fields should be changed)
@thiagodeev Hello! I raised a draft PR to track my progress. qq, can we use the rpc.U64 and rpc.u128 types for this?
Hey @danielcdz, thanks again. Our team has decided internally to stop working on this matter. For now, we will let things as they are. In the future, when we implement Cairo types in starknet.go, we'll make this change.
If you can, there's something else we need help: a method for deploying contracts via UDC (https://github.com/NethermindEth/starknet.go/issues/381)
@thiagodeev Got it, thanks for the heads-up! I’d be more than happy to help with issue #381. I’ll need a bit of context and guidance to get started, but I’m eager to learn and support the team on this