Skip to content
This repository has been archived by the owner on Mar 10, 2023. It is now read-only.

Revert "update amounts to be in uceles not celes" #23

Merged
merged 1 commit into from
May 16, 2022
Merged

Conversation

jbowen93
Copy link
Contributor

This reverts commit c622ba8.

This results in

panic: signature verification failed; please verify account number (0) and chain-id (ephemeral): unauthorized

so we likely need to redo the genesis process entirely.

@evan-forbes
Copy link
Member

may or may not be related to this celestiaorg/celestia-app#379 (comment)

Copy link
Member

@adlerjohn adlerjohn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK, but missing context on why this is needed.

@jbowen93
Copy link
Contributor Author

This is needed because in the previous PR #22 I made an untested last minute change to update the denom values in the genesis.json file from celes to uceles. However I didn't regenerate the genesis static genesis file. Somewhere in this process something is mismatched.

The correct move forward is to redo the whole genesis using after we decide on a denom value we wish to use for Testnet. For now I'd prefer to have the ephemeral cluster working with an "incorrect" denom than not.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants