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

cBRL core contracts deployment #8803

Closed
wants to merge 10 commits into from

Conversation

luisgj
Copy link
Contributor

@luisgj luisgj commented Oct 19, 2021

Description

Adds Proxy and Implementations for the cBRL Exchange and StableToken core smart contracts.

Other changes

Adds migrations support for the cBRL token deployment process.

Tested

Testing ongoing.

Related issues

Backwards compatibility

This is backwards compatible as it does not introduce any changes to any core contract logic but rather just add new ones inheriting from their base implementations.

Documentation

Not sure yet.

Copy link
Contributor

@yorhodes yorhodes left a comment

Choose a reason for hiding this comment

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

Looks great!
We will need to make changes at the SDK level too

@yorhodes
Copy link
Contributor

#8334 fixes the exchange init params issue with the zero address for stableToken address
an additional exchangeBRL.activateStable() transaction from governance will be necessary
cc @martinvol

@martinvol
Copy link
Contributor

We're gonna use #8831 instead

@martinvol martinvol closed this Nov 3, 2021
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.

3 participants