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

Staking Bech/Naming Updates #2221

Closed
rigelrozanski opened this issue Sep 3, 2018 · 3 comments · Fixed by #2251
Closed

Staking Bech/Naming Updates #2221

rigelrozanski opened this issue Sep 3, 2018 · 3 comments · Fixed by #2251
Assignees

Comments

@rigelrozanski
Copy link
Contributor

based on earlier conversations today (CC @jaekwon, @cwgoes, @ValarDragon) - within staking validator object:

  • Operator -> OperatorAddr
  • PubKey -> ConsPubKey

Within the bech prefixes we should rename:

  • cosmoscons -> cosmosvalcons
  • cosmosval -> cosmosvaloper

Or something along those lines - feel free to edit if this has a mistake

(CC @alexanderbez)

@fedekunze
Copy link
Collaborator

Much better @rigelrozanski. Thanks !

@alexanderbez
Copy link
Contributor

Man I'm glad we're making all these changes #prelaunch 😆

@alexanderbez alexanderbez self-assigned this Sep 4, 2018
@cwgoes
Copy link
Contributor

cwgoes commented Sep 4, 2018

Man I'm glad we're making all these changes #prelaunch laughing

Agreed, I don't think we can change these at all post-launch - once integrated into clients / block explorers / HW wallets etc. it will be nearly impossible to revise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants