Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: rollup CoFiX #92

Merged
merged 1 commit into from
Dec 16, 2020
Merged

feat: rollup CoFiX #92

merged 1 commit into from
Dec 16, 2020

Conversation

dekz
Copy link
Member

@dekz dekz commented Dec 16, 2020

Description

There's no old bridge address for CoFiX as we need an ETH fee (which isn't forwarded).
So I just chose 0xc0f1cccccccccccccccccccccccccccccccccccc as the Bridge address for CoFiX

Testing instructions

Types of changes

Checklist:

  • Prefix PR title with [WIP] if necessary.
  • Add tests to cover changes as needed.
  • Update documentation as needed.
  • Add new entries to the relevant CHANGELOG.jsons.

@dekz dekz requested a review from dorothy-zbornak December 16, 2020 23:26
@dekz dekz changed the base branch from development to feat/bancor-fqt-rollup December 16, 2020 23:26
@dekz dekz force-pushed the feat/rollup-cofix branch from 1ab5c62 to 005f4bb Compare December 16, 2020 23:27
@dekz dekz merged commit 650e768 into feat/bancor-fqt-rollup Dec 16, 2020
@dekz dekz deleted the feat/rollup-cofix branch December 16, 2020 23:43
dorothy-zbornak pushed a commit that referenced this pull request Dec 17, 2020
* Add MixinBancor for BridgeAdapter

* Update changelog

* Use 0.6 call value syntax

* `@0x/contract-addresses`: Deploy new FQT

* feat: Rollup CoFiX (#92)

* `@0x/contract-addreses`: update FQT addresses

Co-authored-by: Lawrence Forman <[email protected]>
Co-authored-by: Jacob Evans <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant