Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
SlayerAnsh
left a comment
There was a problem hiding this comment.
I think hard coding denom might not be good idea in longer run specially because osmosis pools have more than 1 fee denom.
We should try to add logic to store the funds send and return any remaining funds after execution.
Motivation
Was checking for exactly 2 funds, but it can also be 3 if uosmo isn't part of the pool assets.
Note: In mainnet the behavior is different, so we can add a conditional check based on the chain_id
Implementation
Testing
The e2e test of pool creation passes
Version Changes
socket-osmosis:0.1.2-b.4->0.1.2-b.5Checklist