-
Notifications
You must be signed in to change notification settings - Fork 78
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
cli: ensure support for every sequencer action type & abci query #1474
Labels
Comments
This was referenced Oct 2, 2024
github-merge-queue bot
pushed a commit
that referenced
this issue
Oct 16, 2024
## Summary Adds Ics20Withdrawal command to the cli for [Ics20Withdrawal](https://buf.build/astria/protocol-apis/docs/main:astria.protocol.transactions.v1alpha1#astria.protocol.transactions.v1alpha1.Ics20Withdrawal) action ## Background part of adding all sequencer actions as cli commands. ## Changes - adds `astria-cli sequencer ics20-withdrawal` command ## Testing - There will be a follow-up PR to add CLI test after the ongoing test refactor is completed. - Tested by performing a manual withdrawal to an IBC chain. Refunds the sender address on failure, which was tested by sending an Ics20Withdrawal with an invalid asset. ## Related Issues part of #1474 closes #1623 --------- Co-authored-by: Sam Bukowski <[email protected]>
github-merge-queue bot
pushed a commit
that referenced
this issue
Nov 6, 2024
## Summary adds ibc sudo address change command ## Background Cli currently does not support the `IbcSudoChange` action ## Changes - added `IbcSudoChange` command to the cli as sudo command ## Testing tested manually against dusk-11 network ## Related Issues part of #1474
github-merge-queue bot
pushed a commit
that referenced
this issue
Nov 6, 2024
Summary Adds bridge-sudo-change command for changing bridge privileges addresses Background Part of adding more actions to the cli Changes - adds `bridge-sudo-change` command setting bridge account privileges addresses - adds `bridge-account get` command querying bridge account information Testing Locally against a cluster Related Issues Part of #1474 closes #1547
This issue is stale because it has been open 45 days with no activity. Remove stale label or this issue |
This issue was closed because it was stale |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently
astria-cli
doesn't support all of our action types which is a hindrance to running manual and adding smoke tests. We want to support all action types, and abci query such that they are all easily tested through our domain types logic.┆Issue Number: ENG-804
The text was updated successfully, but these errors were encountered: