[codex] Add ERC20 lock boost provider#822
Conversation
712c919 to
0407b20
Compare
0407b20 to
a8f0f49
Compare
f8c45db to
bd8b09d
Compare
| /// @notice Initialize the provider. | ||
| /// @param admin Address to receive DEFAULT_ADMIN_ROLE. | ||
| /// @param lockPeriod Initial token lock period. | ||
| function initialize(address admin, uint256 lockPeriod) external; |
There was a problem hiding this comment.
I think we have never included initialize in the interface
There was a problem hiding this comment.
Well, we did it a few times. Make a todo to add others to the interfaces. To keep things consistent
8f72af6 to
d7c0102
Compare
74ec710 to
6a29821
Compare
d7c0102 to
7ccca20
Compare
6a29821 to
02b35d6
Compare
7ccca20 to
6135de9
Compare
02b35d6 to
3d84c2d
Compare
6135de9 to
dd1b86c
Compare
3d84c2d to
7c5bbc7
Compare
dd1b86c to
ed021d3
Compare
7c5bbc7 to
ff17e9e
Compare
ed021d3 to
19dca86
Compare
ff17e9e to
16bedaa
Compare
19dca86 to
5df38bc
Compare
16bedaa to
ea37008
Compare
5df38bc to
935a227
Compare
ea37008 to
07eb55e
Compare
935a227 to
71dec95
Compare
71dec95 to
b342310
Compare
b342310 to
0a0c6c2
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0a0c6c2f14
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| // LDO lock boost provider | ||
| config.ldoLockBoostProviderConfig.token = 0xEf2573966D009CcEA0Fc74451dee2193564198dc; | ||
| config.ldoLockBoostProviderConfig.votingContract = 0x49B3512c44891bef83F8967d075121Bd1b07a01B; | ||
| config.ldoLockBoostProviderConfig.snapshotDelegation = address(1); // TODO: Fill in before deployment |
There was a problem hiding this comment.
Replace the Hoodi snapshot placeholder before deploying
In Hoodi deployments this placeholder is copied into the Lido lock vault implementation as the immutable Snapshot delegation registry address, and every vault created from that beacon will call it from assignSnapshotDelegate/unassignSnapshotDelegate. With address(1) deployed, Hoodi operators cannot actually delegate their locked LDO voting power for Snapshot without a vault implementation upgrade, so the deploy script should use the real Hoodi delegation registry or fail fast until it is known.
Useful? React with 👍 / 👎.
Summary