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

Feat: Cosmos SDK 47 #109

Merged
merged 87 commits into from
Dec 22, 2023
Merged

Feat: Cosmos SDK 47 #109

merged 87 commits into from
Dec 22, 2023

Conversation

pr0n00gler
Copy link
Collaborator

@pr0n00gler pr0n00gler commented Sep 7, 2023

This PR makes neutron-sdk compatible with Neutron V2 release:

@pr0n00gler pr0n00gler dismissed stale reviews from sotnikov-s and swelf19 via d280443 December 14, 2023 17:17
swelf19
swelf19 previously approved these changes Dec 18, 2023
pub fn submit_create_denom(subdenom: impl Into<String>) -> Self {
NeutronMsg::CreateDenom {
subdenom: subdenom.into(),
}
}

// Basic helper to define change of admin for a token passed to TokenFactory module:
Copy link
Contributor

Choose a reason for hiding this comment

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

Does not look clear, what exactly we pass to TokenFactory module. Did you miss the message word?
And the same with alomost every "Basic helper to"

Copy link
Contributor

@ratik ratik left a comment

Choose a reason for hiding this comment

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

please consider changing create_stargate_msg

ratik
ratik previously approved these changes Dec 20, 2023
@sotnikov-s
Copy link
Contributor

@sotnikov-s sotnikov-s requested review from ratik and swelf19 December 20, 2023 17:58
@pr0n00gler pr0n00gler merged commit fbf2270 into main Dec 22, 2023
14 checks passed
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.

6 participants