Skip to content
This repository was archived by the owner on Oct 15, 2024. It is now read-only.

feat: Add Balancer as a source#277

Closed
moodlezoup wants to merge 2 commits intomasterfrom
feat/balancer
Closed

feat: Add Balancer as a source#277
moodlezoup wants to merge 2 commits intomasterfrom
feat/balancer

Conversation

@moodlezoup
Copy link
Copy Markdown
Contributor

@moodlezoup moodlezoup commented Jul 10, 2020

Updates @0x/asset-swapper and @0x/contract-addresses to add Balancer support.
Reflects updates to the ERC20BridgeSource enum and the new gas/fee schedule interface.
DO NOT MERGE until

  • BalancerBridge 0x-monorepo#2613 has been approved & merged
  • @0x/asset-swapper and @0x/contract-addresses are updated to non-gitpkg versions, or at least gitpkg publishes that are on the monorepo development branch
  • simbot comparing staging vs prod looks good

Description

Testing Instructions

Checklist

  • Update documentation as needed. Website Documentation PR:
  • Prefix PR title with [WIP] if necessary.
  • Add tests to cover changes as needed.

[ERC20BridgeSource.Kyber]: () => 8e5,
[ERC20BridgeSource.Curve]: fillData => {
switch ((fillData as CurveFillData).poolAddress.toLowerCase()) {
case '0xa2b47e3d5c44877cca798226b7b8118f9bfb7a56':
Copy link
Copy Markdown
Contributor

@xianny xianny Jul 10, 2020

Choose a reason for hiding this comment

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

can we make these addresses constants somewhere? Maybe even just an obj in this file.

CurvePoolAddresses = {
    UsdcDai: '...',
}

@moodlezoup moodlezoup closed this Jul 20, 2020
@xianny xianny deleted the feat/balancer branch March 30, 2021 22:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants