Skip to content

fix: blockHash parameter casing in eth_getLogs documentation#15534

Merged
wackerow merged 3 commits into
ethereum:devfrom
chainide-agent-bob:fix/json-rpc-doc-eth-getlogs-block-hash-parameter
Jul 16, 2025
Merged

fix: blockHash parameter casing in eth_getLogs documentation#15534
wackerow merged 3 commits into
ethereum:devfrom
chainide-agent-bob:fix/json-rpc-doc-eth-getlogs-block-hash-parameter

Conversation

@chainide-agent-bob
Copy link
Copy Markdown
Contributor

Description

This PR fixes the inconsistent casing of the blockHash parameter in the eth_getLogs JSON-RPC method documentation. The parameter was incorrectly written as blockhash (lowercase) in the documentation, while the EIP-234 specification defines it as blockHash (camelCase).

The changes have been applied to all language versions of the documentation to maintain consistency with the EIP specification.

Related Issue

Fixes #15519

@netlify
Copy link
Copy Markdown

netlify Bot commented May 29, 2025

Deploy Preview for ethereumorg failed.

Name Link
🔨 Latest commit 6067634
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/687833c5593d40000816171d

@github-actions github-actions Bot added content 🖋️ This involves copy additions or edits translation 🌍 This is related to our Translation Program labels May 29, 2025
@github-actions
Copy link
Copy Markdown
Contributor

This issue is stale because it has been open 30 days with no activity.

@github-actions github-actions Bot added the Status: Stale This issue is stale because it has been open 30 days with no activity. label Jun 29, 2025
wackerow and others added 2 commits July 16, 2025 16:20
Co-Authored-By: Bob <203082506+chainide-agent-bob@users.noreply.github.com>
Copy link
Copy Markdown
Member

@wackerow wackerow left a comment

Choose a reason for hiding this comment

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

Looks good! Reverted non-English versions as mentioned in other PR reviews... we generally shouldn't edit the non-English files directly from the repo; they should sync through Crowdin so our translation management system stays up-to-date. Pulling in!

(Those changes will get propagated to the other languages within Crowdin before being synced back each month)

@wackerow wackerow merged commit 17f7db2 into ethereum:dev Jul 16, 2025
3 checks passed
This was referenced Jul 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content 🖋️ This involves copy additions or edits Status: Stale This issue is stale because it has been open 30 days with no activity. translation 🌍 This is related to our Translation Program

Projects

None yet

Development

Successfully merging this pull request may close these issues.

blockhash in eth_getLogs should be blockHash according to eip-234

2 participants