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

Script to generate change registration governance #1508

Merged
merged 3 commits into from
Sep 6, 2022

Conversation

bruce-riley
Copy link
Contributor

@bruce-riley bruce-riley commented Sep 2, 2022

Change-Id: If99e78b031dcd4049a9ee40b09914eeb93e7518a

Also changing the guardian verify VAA command to print the digest in the same format as the execute VAA command.

Much of the magic in register-chain-governance.sh was taken directly from contract-upgrade-governance.sh, which has previously been reviewed.

Change-Id: If99e78b031dcd4049a9ee40b09914eeb93e7518a
clients/js/main.ts Outdated Show resolved Hide resolved
clients/js/main.ts Outdated Show resolved Hide resolved
panoel
panoel previously approved these changes Sep 2, 2022
Copy link
Contributor

@kcsongor kcsongor left a comment

Choose a reason for hiding this comment

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

This is awesome!

clients/js/main.ts Outdated Show resolved Hide resolved
Change-Id: I93cf3166c1ab961add2c524084de1981a44b194e
kcsongor
kcsongor previously approved these changes Sep 5, 2022
clients/js/main.ts Outdated Show resolved Hide resolved
Change-Id: Iae6f9854a64d8559d03c56949d03043a94dbf7f6
@bruce-riley bruce-riley merged commit b357ad7 into dev.v2 Sep 6, 2022
@bruce-riley bruce-riley deleted the register_chain_governance_script branch September 6, 2022 13:20
evan-gray pushed a commit that referenced this pull request Sep 7, 2022
* Script to generate change registration governance

Change-Id: If99e78b031dcd4049a9ee40b09914eeb93e7518a

* Rework from code review

Change-Id: I93cf3166c1ab961add2c524084de1981a44b194e

* Change chain_id to chain-id

Change-Id: Iae6f9854a64d8559d03c56949d03043a94dbf7f6
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.

3 participants