-
Notifications
You must be signed in to change notification settings - Fork 463
feat: Add CREAM as a liquidity source #2715
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need an entry in contract-addresses
for the creamBridge
to keep it stored somewhere?
Probably needed for the migrations
as well.
contracts/zero-ex/contracts/src/transformers/bridges/BridgeAdapter.sol
Outdated
Show resolved
Hide resolved
packages/asset-swapper/src/utils/market_operation_utils/sampler_operations.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small nits, but looking gr8!
const MAX_POOLS_FETCHED = 3; | ||
// tslint:enable:custom-no-magic-numbers | ||
|
||
export class CreamPoolsCache { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there really any meaningful difference between this class and BalancerPoolsCache
? Does it make sense to just modify and inherit from that class, passing in getPoolsWithTokens()
and parsePoolData()
callbacks to the super?
packages/asset-swapper/src/utils/market_operation_utils/index.ts
Outdated
Show resolved
Hide resolved
packages/asset-swapper/src/utils/market_operation_utils/index.ts
Outdated
Show resolved
Hide resolved
packages/asset-swapper/src/utils/market_operation_utils/types.ts
Outdated
Show resolved
Hide resolved
packages/asset-swapper/src/utils/market_operation_utils/orders.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Think this might need a rebase ontop of latest development, then a re-publish once on development
for 0xAPI. So that the sha
matches the commit.
Once this is merged github basically deletes the sha
making it hard to find which AS release was which commit, unless published off development.
So rebase, merge, publish:private
Description
Add CREAM to asset-swapper.
CREAM (Crypto Rules Everything Around Me) is a Balancer clone.
Testing instructions
Types of changes
Checklist:
[WIP]
if necessary.