Malte Kliemann

Migrate neo-swaps pools to bounded storage

1

### What does it do? Migrates neo-swaps pools from `BTreeMap` to `BoundedBTreeMap` to avoid potentially faulty manual `MaxEncodedLen` implementations. Fixes #1214. ### What important points should reviewers know? ### Is...

Oracle duration doesn't make sense for trusted markets

1

Trusted markets don't allow outsider reports, so why even configure the dispute period?

Add integration tests for RPC calls and runtime configuration

5

We've had some RPC calls fail due to incorrect order of arguments and encoding problems, which can't easily be caught. Writing some integration tests might help here. Also, @vivekvpandya pointed...

The implementation directly calls `do_deploy_pool`, which is tested via the pallet's extrinsic interface, but it would be prudent to go the extra mile and test `DeployPoolApi` directly. This means figuring...

[Court] Can't partially reduce my stake

2

It seems like I can't reduce my stake - I have to unstake everything. Why? That's very confusing.

Concern have been raised that the liquidity tree may hurt adoption is small LPs are blocked from joining the pool by whales due to the requirement that each new LP...

Instead, we should just use whitelists - but frame-system doesn't offer that functionality yet.