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

README.md: Update celes to uceles #210

Merged
merged 2 commits into from
Feb 21, 2022

Conversation

renaynay
Copy link
Member

This PR updates the README.md to use the current denom which is uceles instead of celes.

tac0turtle
tac0turtle previously approved these changes Feb 11, 2022
adlerjohn
adlerjohn previously approved these changes Feb 11, 2022
readme.md Outdated
Comment on lines 14 to 15
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
Copy link
Member

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

liamsi
liamsi previously approved these changes Feb 19, 2022
Copy link
Member

@evan-forbes evan-forbes left a 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

readme.md Outdated Show resolved Hide resolved
Co-authored-by: Evan Forbes <42654277+evan-forbes@users.noreply.github.com>
@evan-forbes evan-forbes merged commit 6e0140b into celestiaorg:master Feb 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants