-
Notifications
You must be signed in to change notification settings - Fork 208
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
ICS loose ends #881
ICS loose ends #881
Conversation
|
1 similar comment
|
@strbrian can you add a more detailed description to this PR? |
Updated the PR description. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good - we also discussed registering new denoms when a new host zone is added.
How are you thinking this will work?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm! when we start dockernet, stuatom should now be registered as a fee denom - let's verify that as we test the upgrade
exactly, I actually already verified that as well. |
Context and purpose of the change
This PR involves whitelisting missing param keys and a helper function for registering stToken denoms to consumer whitelist.
stTokens need to be whitelisted on consumer chain to be flown to provider chain and distributed to hub validators.
These updates were originally included into this PR but we pulled ICS updates from the PR for the convenience.
Brief Changelog
Author's Checklist