Skip to content

Fix uniswap for celo#1065

Merged
0xh3rman merged 1 commit intomainfrom
celo-swap-fix
Apr 9, 2026
Merged

Fix uniswap for celo#1065
0xh3rman merged 1 commit intomainfrom
celo-swap-fix

Conversation

@0xh3rman
Copy link
Copy Markdown
Collaborator

@0xh3rman 0xh3rman commented Apr 9, 2026

No description provided.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a new native_asset module to manage tokenized native assets, specifically for Celo, by implementing uses_msg_value and is_tokenized_native functions. This new logic is integrated into the Uniswap V3 and V4 swapper providers and command builders. The changes also include adding constants for Celo USDT and WETH (WCELO) and updating their usage across the codebase. Review feedback indicates that the CELO_USDT_TOKEN_ID constant is misnamed and should be updated to reflect 'Moola cUSD' (mcUSD), and CELO_WETH_TOKEN_ID should be renamed to CELO_WCELO_TOKEN_ID for better clarity, as it refers to Wrapped Celo.

Comment thread crates/primitives/src/asset_constants.rs
Comment thread crates/primitives/src/asset_constants.rs
@0xh3rman 0xh3rman merged commit 174aa30 into main Apr 9, 2026
5 checks passed
@0xh3rman 0xh3rman deleted the celo-swap-fix branch April 9, 2026 12:31
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.

1 participant