White-Whale-Defi-Platform / white-whale-core

Core contracts for White Whale, The Interchain Liquidity Protocol for the Cosmos.
https://whitewhale.money
MIT License
36 stars 23 forks source link

Feat/remove cw20 from pool manager #299

Closed nseguias closed 5 months ago

nseguias commented 5 months ago

Description and Motivation

This PR removes all cw20 implementations from the pool manager. Once merged, the Pool Manager will no longer support CW20 tokens, it will only work using token factory tokens.

Related Issues


Checklist:

codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 79.70149% with 68 lines in your changes are missing coverage. Please review.

Project coverage is 88.50%. Comparing base (e59999f) to head (b3cde00). Report is 2 commits behind head on release/v2_contracts.

Files Patch % Lines
...quidity_hub/pool-manager/src/tests/mock_querier.rs 0.00% 43 Missing :warning:
...acts/liquidity_hub/pool-manager/src/tests/suite.rs 69.23% 8 Missing :warning:
...ontracts/liquidity_hub/pool-manager/src/queries.rs 50.00% 5 Missing :warning:
packages/white-whale-std/src/pool_manager.rs 55.55% 4 Missing :warning:
...ts/liquidity_hub/pool-manager/src/swap/commands.rs 80.00% 3 Missing :warning:
...ty_hub/pool-manager/src/tests/integration_tests.rs 97.54% 3 Missing :warning:
...ntracts/liquidity_hub/pool-manager/src/contract.rs 83.33% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## release/v2_contracts #299 +/- ## ======================================================== - Coverage 88.50% 88.50% -0.01% ======================================================== Files 252 252 Lines 24029 23604 -425 ======================================================== - Hits 21267 20890 -377 + Misses 2762 2714 -48 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

kaimen-sano commented 5 months ago

CI also needs to be resolved.