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.

Sign up for GitHub

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

Jump to bottom

Voting for stETH on Optimism #245

Merged
merged 29 commits into from
Sep 30, 2024
Merged

Voting for stETH on Optimism #245

merged 29 commits into from
Sep 30, 2024

Conversation

arwer13
Copy link
Contributor

@arwer13 arwer13 commented Aug 23, 2024 •
edited by kovalgek
Loading

WHAT

Vote for upgrading stETH on Optimism + two fallback scripts

TODO

  • texts
    • Top text comment
    • Description
    • vote description items
  • broken tests
    • test_simple_dvt_module
    • test_all_round_happy_path
    • test_el_rewards
    • test_sdvt_rewards_happy_path
    • test_staking_router_stake_distribution
    • test_voting

loga4
loga4 reviewed Sep 16, 2024
View reviewed changes
tests/snapshot/test_dsm.py Outdated Show resolved Hide resolved
@kovalgek kovalgek requested a review from loga4 September 23, 2024 08:52
@kovalgek kovalgek marked this pull request as ready for review September 23, 2024 10:54
@kovalgek kovalgek requested a review from TheDZhon September 23, 2024 12:15
@Ivan-Feofanov Ivan-Feofanov force-pushed the feat/vote-steth-optimism branch from d97c238 to 0d8f8d9 Compare September 26, 2024 11:41
kovalgek and others added 24 commits September 26, 2024 13:42
@kovalgek @Ivan-Feofanov
upgrade L1 optimism bridge
dbbf25a
@kovalgek @Ivan-Feofanov
add optimism contracts upgrade actions
b181576
@arwer13 @Ivan-Feofanov
feat: mvp code to run the voting on Sepolia
1e712fe
@kovalgek @Ivan-Feofanov
fix l2 actions, add function to run vote for fork
e818d6a
@kovalgek @Ivan-Feofanov
add upgrade finalization to l1, l2 bridges and l2 wstETH
5298b0d
@kovalgek @Ivan-Feofanov
add tests
b979169
@kovalgek @Ivan-Feofanov
add tests for oracle report and token rate notifier
8e7a763
@kovalgek @Ivan-Feofanov
fix small comments
e51378f
@kovalgek @Ivan-Feofanov
rename files
e5b453c
@kovalgek @Ivan-Feofanov
disable tests for mainnet that are for sepolia state only, rename files
4277d8f
@arwer13 @Ivan-Feofanov
feat(stETH on optimism): add fallback vote script for enabling deposits
c7375e1
@arwer13 @Ivan-Feofanov
feat(stETH on optimism): add mvp of the upgrade script
56b9b02
@arwer13 @Ivan-Feofanov
feat(stETH on optimism): add fallback vote script if L2 upgrade fails
1598211
@TheDZhon @Ivan-Feofanov
fix: import brownie to mitigate reverts failure
f8ac524
@kovalgek @Ivan-Feofanov
fix voting for executing on forks
5765220
@kovalgek @Ivan-Feofanov
top up contract balances for london hardfork
16537ad
@kovalgek @Ivan-Feofanov
top up balances in fixtures
abdaa2d
@kovalgek @Ivan-Feofanov
use middleware to top up balances
d752529
@kovalgek @Ivan-Feofanov
fix typo
befebcb
@kovalgek @Ivan-Feofanov
fix constants
126a1f1
@kovalgek @Ivan-Feofanov
update description and voting text
1cb3aad
@kovalgek @Ivan-Feofanov
fix test by considering gas usage in tx
4568af1
@kovalgek @Ivan-Feofanov
add pause and resume help methods
b27f41f
@kovalgek @Ivan-Feofanov
fix el reward test by considering gas usage in tx
77fd946
@Ivan-Feofanov Ivan-Feofanov force-pushed the feat/vote-steth-optimism branch from 0d8f8d9 to 7eae83f Compare September 26, 2024 13:26
@Ivan-Feofanov Ivan-Feofanov changed the base branch from master to feat/next-vote September 30, 2024 09:12
@Ivan-Feofanov Ivan-Feofanov self-requested a review September 30, 2024 09:17
@Ivan-Feofanov Ivan-Feofanov merged commit a44e546 into feat/next-vote Sep 30, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@loga4 loga4 loga4 left review comments

@Ivan-Feofanov Ivan-Feofanov Ivan-Feofanov approved these changes

@TheDZhon TheDZhon Awaiting requested review from TheDZhon

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@arwer13 @loga4 @Ivan-Feofanov @kovalgek @TheDZhon