-
Notifications
You must be signed in to change notification settings - Fork 386
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
README.md: Update celes
to uceles
#210
Conversation
readme.md
Outdated
celestia-appd add-genesis-account <address from above command> 10000000celes,1000token | ||
celestia-appd gentx user1 100000celes --chain-id test | ||
celestia-appd add-genesis-account <address from above command> 10000000uceles,1000token | ||
celestia-appd gentx user1 100000uceles --chain-id test |
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.
when we gentx here, the staking module will through a weird error here unless there is at least 1celes (1000000uceles) being used as stake
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.
running these commands as is doesn't actually work, we need to add an extra digit when we gentx
.
should have been changed a long time ago 😬, but I think it makes sense to change while we are fixing README instructions
Co-authored-by: Evan Forbes <42654277+evan-forbes@users.noreply.github.com>
6831ed5
This PR updates the
README.md
to use the current denom which isuceles
instead ofceles
.