Skip to content
This repository has been archived by the owner on Mar 24, 2023. It is now read-only.

feat: add blockSpecifer for blockParameter to support Eip1898 #476

Merged
merged 2 commits into from
Aug 4, 2022

Conversation

RetricSu
Copy link
Contributor

@RetricSu RetricSu commented Jul 29, 2022

{
    "jsonrpc": "2.0",
    "method": "eth_call",
    "params": [
        {
            "from": "0x966b30e576a4d6731996748b48dd67c94ef29067",
            "to": "0x45b2585ee6877b36e359603eae3c465392b57d13",
            "data": "0xb67d77c500000000000000000000000000000000000000000000000000000000000000080000000000000000000000000000000000000000000000000000000000000006"
        },
        {
            "blockHash": "0x65b3f50d57e73112852d926f5f18cd5714a487c81ed9da1b9a1a9bd16a9e2d81",
            "requireCanonical": false
        }
    ],
    "id": 1
}

{
    "jsonrpc": "2.0",
    "id": 1,
    "result": "0x0000000000000000000000000000000000000000000000000000000000000002"
}

References

Test

@RetricSu RetricSu linked an issue Jul 29, 2022 that may be closed by this pull request
@RetricSu RetricSu requested review from keroro520, classicalliu and Flouse and removed request for keroro520 July 29, 2022 10:31
Copy link
Contributor

@keroro520 keroro520 left a comment

Choose a reason for hiding this comment

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

Please add some test cases (https://eips.ethereum.org/EIPS/eip-1898 has listsed test cases).

@RetricSu RetricSu merged commit 1a5326e into main Aug 4, 2022
@RetricSu RetricSu deleted the eip-1898 branch August 4, 2022 02:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

eip-1898 compatibility (for graph-node)
3 participants