Skip to content

feat: mRe7ETH on Optimism#199

Merged
dmytro-horbatenko merged 1 commit intomainfrom
feat/mre7eth-on-optimism
Apr 1, 2026
Merged

feat: mRe7ETH on Optimism#199
dmytro-horbatenko merged 1 commit intomainfrom
feat/mre7eth-on-optimism

Conversation

@dmytro-horbatenko
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces support for the Optimism network and implements the mRe7ETH token ecosystem, including its associated custom aggregator feeds, data feeds, and vault contracts. The changes encompass extensive configuration updates for network parameters, contract addresses, and deployment scripts. Feedback identifies critical placeholder 'dummy' addresses in the mRe7ETH deployment configuration that must be updated before production use, as well as a minor formatting issue regarding a missing newline in the roles documentation.

Comment on lines +36 to +38
liquidityProvider: 'dummy',
enableSanctionsList: true,
swapperVault: 'dummy',
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

critical

The liquidityProvider and swapperVault properties are set to 'dummy'. These placeholder values must be replaced with the correct addresses before deployment to ensure the redemption vault swapper functions correctly.

| ***pauser*** | `0x8f4b7dfb05ebffc760b8d398af5b0d327b890a22f9030321740d68be514e8f60` |
| ***customFeedAdmin*** | `0xc9f03f672dadad915001ad6a5393aba8482f9c28ea275c4b5480f7b994778ed5` |
| ***depositVaultAdmin*** | `0x798de9824ed446b72c97afa7b28844dbbfbde8830d58977e9462289a757aafb7` |
| ***redemptionVaultAdmin*** | `0x37c36a61c043b91d5ec6d117c64ef865031d25c49609bb25d6530b1fcb8e2e1c` | No newline at end of file
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

This file is missing a final newline. Please add one to adhere to standard file formatting conventions.

@dmytro-horbatenko dmytro-horbatenko merged commit 64ad900 into main Apr 1, 2026
4 checks passed
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.

2 participants