-
Notifications
You must be signed in to change notification settings - Fork 608
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
refactor/test(x/cosmwasmpool): tokenfactory shares and updated transmuter #5381
Conversation
// For choice, see: https://github.com/osmosis-labs/osmosis/pull/4983 | ||
DenomCreationFee: sdk.NewCoins(), // used to be 10 OSMO at launch. | ||
DenomCreationGasConsume: uint64(DefaultCreationGasFee), |
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.
See discussion for context: https://osmosis-network.slack.com/archives/C04LZFBHGQ5/p1685588880275719?thread_ts=1685506174.651869&cid=C04LZFBHGQ5
Also, see this: #4983
I saw the simulator failure and will address it tomorrow. If everything else looks good, please ACK the PR, and I will merge it after fixing the simulator |
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 I will fix share denom
…uter (#5381) * new-transmuter * refactor/test(x/cosmwasmpool): tokenfactory shares and updated transmuter * updates * updates * fix test * disable simulator * update transmuter * correct share * fix test AGAIN * fix test * fix problem * fix * update test * updates * remove
Closes: #XXX
What is the purpose of the change
This PR implements the change documented in: #5350
With the transmuter contract from this branch: https://github.com/osmosis-labs/transmuter/tree/share_token
Testing and Verifying
Functionally tests the transmuter tokenfactory shares.
Documentation and Release Note
Unreleased
section ofCHANGELOG.md
?Where is the change documented?
x/{module}/README.md
)