Skip to content

fix(optimism): Operator Fee Calculation & Deposit Identifier#2257

Merged
rakita merged 4 commits intobluealloy:release/v55from
refcell:rf/deposit-tx-fix
Mar 21, 2025
Merged

fix(optimism): Operator Fee Calculation & Deposit Identifier#2257
rakita merged 4 commits intobluealloy:release/v55from
refcell:rf/deposit-tx-fix

Conversation

@refcell
Copy link
Contributor

@refcell refcell commented Mar 19, 2025

Description

  • Fixes deposit tx identifier to be 0x7E instead of 0x7F.
  • Fixes the operator fee calculation. h/t @leruaa 🎉

Action tests against the golang implementation are demonstrated to pass in op-rs/kona#1277

@refcell refcell force-pushed the rf/deposit-tx-fix branch 2 times, most recently from 2c48bfb to d02925c Compare March 19, 2025 13:35
@refcell refcell changed the title fix(optimism): Operator Fee Calculation fix(optimism): Operator Fee Calculation & Deposit Identifier Mar 19, 2025
Co-authored-by: leruaa <aurelien.catinon@gmail.com>
@refcell refcell force-pushed the rf/deposit-tx-fix branch from d02925c to a3c8aee Compare March 19, 2025 13:37
@refcell refcell marked this pull request as draft March 19, 2025 13:47
@refcell
Copy link
Contributor Author

refcell commented Mar 19, 2025

Looks like there's 1 more action test left to fix before we land these changes

@refcell refcell marked this pull request as ready for review March 20, 2025 21:29
@rakita rakita merged commit 6d27c88 into bluealloy:release/v55 Mar 21, 2025
22 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.

fix(revm): Operator fee ETH mint bug

4 participants