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

Update EIP-7804: fix typos #9190

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

Update EIP-7804: fix typos #9190

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions EIPS/eip-7804.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@
Execution request can be created via smart contracts, allowing for decentralized
and on-chain mechanisms to be explored.
This also means validators will be able to move between execution and compounding
withdrawal credentials (follwoing the correct churn on the total staked amount).
withdrawal credentials (following the correct churn on the total staked amount).

For an execution request to be authorized in the consensus layer, the withdrawal
credential of the validator must be the same of the `msg.caller` when processing
Expand Down Expand Up @@ -116,7 +116,7 @@
2. Excess requests getter - if the input length is zero, return the current excess requests count.
3. System process - if called by system address, pop off the withdrawal credential update requests for the current block from the queue.

<!-- TODO add sudo code / bytecode / deployment / etc (very similar to existing request contracts) -->

Check warning on line 119 in EIPS/eip-7804.md

View workflow job for this annotation

GitHub Actions / EIP Walidator

HTML comments are only allowed while `status` is one of: `Draft`, `Withdrawn` 

warning[markdown-html-comments]: HTML comments are only allowed while `status` is one of: `Draft`, `Withdrawn`
   --> EIPS/eip-7804.md
    |
119 | <!-- TODO add sudo code / bytecode / deployment / etc (very similar to existing request contracts) -->
    |
   ::: EIPS/eip-7804.md
    |
132 | <!-- TODO complete the specification -->
    |
   ::: EIPS/eip-7804.md
    |
195 | <!-- TODO -->
    |
   ::: EIPS/eip-7804.md
    |
199 | <!-- TODO -->
    |
   ::: EIPS/eip-7804.md
    |
203 | <!-- TODO -->
    |
    = help: see https://ethereum.github.io/eipw/markdown-html-comments/

#### Block processing

Expand Down Expand Up @@ -207,7 +207,7 @@
Ownership is defined based on control of the withdrawal credential account, either as a private key (for EOA accounts) or controlling
the smart contract at the address set as withdrawal credential. Therefore allowing an update should not bring any security implications.

Futher discussion needed.
Further discussion needed.

## Copyright

Expand Down
Loading