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

CLI: Allow using key name for --admin #1039

Closed
webmaster128 opened this issue Oct 6, 2022 · 1 comment · Fixed by #1103
Closed

CLI: Allow using key name for --admin #1039

webmaster128 opened this issue Oct 6, 2022 · 1 comment · Fixed by #1103
Assignees
Labels
help wanted Extra attention is needed
Milestone

Comments

@webmaster128
Copy link
Member

webmaster128 commented Oct 6, 2022

The CLI field --from string Name or address of private key with which to sign allows me to use the local key alias. Now when I instantiate a contract with myself as the admin, I would like to do --from my_super_key --admin my_super_key without having to look up the address manually to avoid the risk of using the wrong one. However, when I try to do that I get

Error: admin: decoding bech32 failed: invalid separator index -1

Would that be possible to implement? Does it make sense?

@alpe
Copy link
Contributor

alpe commented Oct 7, 2022

👍 Sounds good to improve UX

@alpe alpe moved this to 🆕 New in wasmd backlog Oct 7, 2022
@alpe alpe moved this from 🆕 New to 📋 Unsorted Backlog in wasmd backlog Oct 7, 2022
@ethanfrey ethanfrey added this to the v0.31.0 milestone Nov 4, 2022
@alpe alpe added the help wanted Extra attention is needed label Nov 15, 2022
@pinosu pinosu self-assigned this Nov 18, 2022
Repository owner moved this from 📋 Unsorted Backlog to ✅ Done in wasmd backlog Dec 6, 2022
@alpe alpe removed this from wasmd backlog Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants