Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: replace ExitV1 with WithdrawalRequest #672

Merged
merged 5 commits into from
May 3, 2024

Conversation

onbjerg
Copy link
Member

@onbjerg onbjerg commented May 3, 2024

Motivation

EIP-7002 has been changed to work with EIP-7685, so the exit type no longer exists and is instead a WithdrawalRequest object.

In the engine API spec this is called WithdrawalRequestV1.

Solution

  • Move ExitV1 to alloy-eips under a new eip7002 module
  • Rename it to WithdrawalRequest
  • Re-export alloy_eips::eip7002::WithdrawalRequest as WithdrawalRequestV1

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

EIP-7002 has been changed to work with EIP-7685,
so the exit type no longer exists and is instead a
`WithdrawalRequest` object.

In the engine API spec this is called `WithdrawalRequestV1`,
which is why it is re-exported as such from `alloy-eips`
@onbjerg onbjerg added the debt Tech debt which needs to be addressed label May 3, 2024
@prestwich prestwich merged commit c9b2a5b into main May 3, 2024
24 checks passed
@prestwich prestwich deleted the onbjerg/eip7002-updates branch May 3, 2024 19:01
ben186 pushed a commit to ben186/alloy that referenced this pull request Jul 27, 2024
* chore: replace `ExitV1` with `WithdrawalRequest`

EIP-7002 has been changed to work with EIP-7685,
so the exit type no longer exists and is instead a
`WithdrawalRequest` object.

In the engine API spec this is called `WithdrawalRequestV1`,
which is why it is re-exported as such from `alloy-eips`

* fix: cfg attr for serde

* chore: correct field name

* chore: cfg attr _-_

* chore: delete `exit` module (unused)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
debt Tech debt which needs to be addressed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants