Closed kaimen-sano closed 3 months ago
Attention: Patch coverage is 95.28302%
with 5 lines
in your changes are missing coverage. Please review.
Project coverage is 89.42%. Comparing base (
64ec60f
) to head (c63be19
). Report is 10 commits behind head on release/v2_contracts.:exclamation: Current head c63be19 differs from pull request most recent head 1412749. Consider uploading reports for the commit 1412749 to get more accurate results
Files | Patch % | Lines |
---|---|---|
...acts/liquidity_hub/pool-manager/src/tests/suite.rs | 80.00% | 3 Missing :warning: |
...ty_hub/pool-manager/src/tests/integration_tests.rs | 97.14% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description and Motivation
UpdateConfig
msg to be sent to thepool-manager
contract to update the contract configuration.QueryMsg::Config
on thepool-manager
contractRelated Issues
closes #294, related to #309.
Before merging this PR, we need to have a think about how we want to handle contract ownership. We currently have the cosmwasm contract admin, the
cw-ownable
owner, and theConfig.owner
.Checklist:
Update index.md
)cargo fmt --all --
.cargo clippy -- -D warnings
.cargo schema
.