Skip to content

Update EIP-6110: harmonize request eips with 7685 changes#8949

Merged
lightclient merged 2 commits intoethereum:masterfrom
lightclient:more-devnet-4
Oct 10, 2024
Merged

Update EIP-6110: harmonize request eips with 7685 changes#8949
lightclient merged 2 commits intoethereum:masterfrom
lightclient:more-devnet-4

Conversation

@lightclient
Copy link
Copy Markdown
Member

This cleans up 6110, 7002, and 7251 with respect to the modifications to 7685. Specifically, request_data now refers to the output of the system call to the contracts, not an individual request object's data.

@lightclient lightclient requested a review from eth-bot as a code owner October 9, 2024 21:07
@github-actions github-actions bot added c-update Modifies an existing proposal s-review This EIP is in Review t-core labels Oct 9, 2024
@eth-bot
Copy link
Copy Markdown
Collaborator

eth-bot commented Oct 9, 2024

File EIPS/eip-6110.md

Requires 1 more reviewers from @djrtwo, @mkalinin, @petertdavies

@eth-bot eth-bot added the a-review Waiting on author to review label Oct 9, 2024
@eth-bot eth-bot changed the title harmonize request eips with 7685 changes Update EIP-6110: harmonize request eips with 7685 changes Oct 9, 2024
@@ -73,11 +73,9 @@ Note that `amount` is returned by the contract little-endian, and must be encode

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The EIP-7685 encoding of a withdrawal request is computed as follows.

Can you also fix the above link to [EIP-7685](./eip-7685.md)? It is missing the md extension

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

fixed. thx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a-review Waiting on author to review c-update Modifies an existing proposal s-review This EIP is in Review t-core

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants