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.

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

Update EIP-7702: Update description #9228

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Thegaram
Copy link

@Thegaram Thegaram commented Jan 9, 2025

The word permanently was added in 52c5677 but this seems to be outdated. In the latest version of EIP-7702, SET_CODE_TX_TYPE transactions can set, update, or clear an account's delegation designation.

@Thegaram Thegaram requested a review from eth-bot as a code owner January 9, 2025 09:59
@Thegaram Thegaram changed the title Update EIP-7702> description Update EIP-7702: Update description Jan 9, 2025
@github-actions github-actions bot added c-update Modifies an existing proposal s-review This EIP is in Review t-core labels Jan 9, 2025
@eth-bot
Copy link
Collaborator

eth-bot commented Jan 9, 2025

File EIPS/eip-7702.md

Requires 1 more reviewers from @adietrichs, @lightclient, @SamWilsn, @vbuterin

@eth-bot eth-bot added the a-review Waiting on author to review label Jan 9, 2025
@lightclient
Copy link
Member

Thanks for this. I added this specifically because people felt it was less clear that 7702 added the delegation permanently. When it was originally proposed it was a temporary set code operation.

@Thegaram
Copy link
Author

Thegaram commented Jan 9, 2025

people felt it was less clear that 7702 added the delegation permanently

Makes sense. But the word permanently suggests that the delegation cannot be changed/removed, at least that would be my interpretation. Maybe persistently (i.e. stored in state) would be more accurate.

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