search

hyperledger / iroha

Iroha - A simple, enterprise-grade decentralized ledger
https://wiki.hyperledger.org/display/iroha
Apache License 2.0
433 stars 277 forks source link

revise query API on client #4892

Open mversic opened 1 month ago

mversic commented 1 month ago 
          I don't like it that on the http client we have API like `asset::all_definitions()`. Because it differs from what we have in smart contracts. IMO there is nothing wrong in constructing a query like this:
    let vec = client
        .query(FindAllAssetDefinitions::new())

Definitely not intended for this PR, but would like to get an opinion on this

_Originally posted by @mversic in https://github.com/hyperledger/iroha/pull/4833#discussion_r1684052028_

mversic commented 1 month ago

in any case API should be the same in client and smart contract