-
Notifications
You must be signed in to change notification settings - Fork 720
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
Add API support for extra node queries #2843
Conversation
This depends on IntersectMBO/ouroboros-network#3220 and should be rebased once that PR is merged to master, so we select a commit from the master branch. |
e9b6472
to
87cccee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Extended the QueryUTxO with a QueryUTxOFilter param which has a new case for QueryUTxOByTxIn. Added QueryStakePools and QueryStakePoolParameters. Added top level query QuerySystemStart Rewind the plutus repo slightly, to match the version being used by the ledger and network repo packages.
Simplify the types by reusing the API QueryUTxOFilter type.
By new --tx-in flag.
Its convenient in tiny testnets but insanely huge on the mainnet, or even the longer running testnets. It's unhelpful to default to that. So we now require using the --whole-utxo flag.
31a22db
to
9192e28
Compare
Rebased on master and updated to refer to the commit from the master branch of the network repo, now that IntersectMBO/ouroboros-network#3220 is merged. |
bors merge |
Build succeeded: |
Extended the QueryUTxO with a QueryUTxOFilter param which has a new case
for QueryUTxOByTxIn.
Added QueryStakePools and QueryStakePoolParameters.
Added top level query QuerySystemStart
Also extend the
cardano-cli query utxo
with a--tx-in
flag to filter by txin, and change the default from returning the whole utxo to there being no default. Added a--whole-utxo
flag for the case that you really do what the whole thing.This change may affect some examples and scripts.