-
Notifications
You must be signed in to change notification settings - Fork 21
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
Make --state-root-hash optional and query the newest one automatically when not provided #79
Comments
after some discussion, the engineering team is not aligned on doing this work. The objective is to remove the awkward way we currently get info out of the client. This was one idea - we will meet to propose alternatives. |
As discussed with @bpr I checked all the methods and listem them below:
|
can't solve in 1.X, can be solved in 2.0 cc: @darthsiroftardis |
The casper-client is inconsistent, for the time being, I want to fix the 3 inconsistent end points and provide the state root hash in the response, to avoid confusion. With 2.0, and update to the client will be required once the RPC is changed. Please proceed with the modifications. |
For example when I'm running
casper-client get-dictionary-item
the--state-root-hash
is required which sometimes is kind of irritating that I need to get the newest one by running another command.We can make it optional and let the user know what happens under the hood.
Make sense @darthsiroftardis @Fraser999 @bpr ? :)
The text was updated successfully, but these errors were encountered: