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

Fix genesis.json stake to celes #158

Closed
4 tasks
Bidon15 opened this issue Dec 2, 2021 · 3 comments · Fixed by #194 or #196
Closed
4 tasks

Fix genesis.json stake to celes #158

Bidon15 opened this issue Dec 2, 2021 · 3 comments · Fixed by #194 or #196
Labels
bug Something isn't working

Comments

@Bidon15
Copy link
Member

Bidon15 commented Dec 2, 2021

Summary of Bug

Now genesis.json contains stake instead of celes as default.

Version

1fcccc4

Mitigation

Manually adjust genesis.json file in places like

cat $genesis_path | jq --arg token $token '.app_state.crisis.constant_fee.denom = $token' | sponge $genesis_path
cat $genesis_path | jq --arg token $token '.app_state.mint.params.mint_denom = $token' | sponge $genesis_path
cat $genesis_path | jq --arg token $token '.app_state.staking.params.bond_denom = $token' | sponge $genesis_path

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@liamsi
Copy link
Member

liamsi commented Jan 3, 2022

Note that there is also a bug in the current devnet where uceles are also called celes.

@liamsi liamsi added the bug Something isn't working label Jan 3, 2022
@tac0turtle
Copy link
Contributor

this is purposefully setup this way in the sdk. There should be a workaround that isn't supported natively but the reason its like this is so the team changing the params needs go through the genesis file manually as its a delicate process.

@evan-forbes evan-forbes moved this to TODO in Celestia Node Jan 13, 2022
@tac0turtle
Copy link
Contributor

I can knock this out quickly.

Repository owner moved this from TODO to Done in Celestia Node Jan 24, 2022
@liamsi liamsi linked a pull request Jan 30, 2022 that will close this issue
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants