Skip to content
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

Improve support authority-gated/gov proposal commands in AutoCLI #18376

Closed
Tracked by #18310
julienrbrt opened this issue Nov 6, 2023 · 0 comments · Fixed by #18461
Closed
Tracked by #18310

Improve support authority-gated/gov proposal commands in AutoCLI #18376

julienrbrt opened this issue Nov 6, 2023 · 0 comments · Fixed by #18461
Assignees

Comments

@julienrbrt
Copy link
Member

julienrbrt commented Nov 6, 2023

Authority gated commands are currently supported, but do not give a great UX as it will by default just broadcast the message. This is unpractical for commands that are expected to be governance/group proposals. We should add an option to RpcCommandOptions commands to signal a command is supposed to be a proposal and have AutoCLI request the necessary proposal fields as well and broadcast a proposal instead of the message directly.

ref: #18310

@julienrbrt julienrbrt self-assigned this Nov 6, 2023
@github-actions github-actions bot added the needs-triage Issue that needs to be triaged label Nov 6, 2023
@julienrbrt julienrbrt added T: UX T: Client UX T:Sprint and removed needs-triage Issue that needs to be triaged labels Nov 6, 2023
@github-project-automation github-project-automation bot moved this to 👀 To Do in Cosmos-SDK Nov 6, 2023
@julienrbrt julienrbrt changed the title Support authority-gated/gov proposal commands: Improve support authority-gated/gov proposal commands in AutoCLI Nov 6, 2023
@github-project-automation github-project-automation bot moved this from 👀 To Do to 🥳 Done in Cosmos-SDK Nov 15, 2023
@tac0turtle tac0turtle removed this from Cosmos-SDK Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants