Skip to content

Handle earliest tag for BlockNumber#836

Merged
sorpaas merged 4 commits intopolkadot-evm:masterfrom
moonbeam-foundation:tgm-bn-earliest
Sep 8, 2022
Merged

Handle earliest tag for BlockNumber#836
sorpaas merged 4 commits intopolkadot-evm:masterfrom
moonbeam-foundation:tgm-bn-earliest

Conversation

@tgmichel
Copy link
Contributor

eth_getLogs and eth_getFilterLogs rpc requests using the earliest tag currently fallback to the best block, which usually returns a misleading empty result. This PR changes that, and handles earliest as the genesis block.

@tgmichel tgmichel requested a review from sorpaas as a code owner August 31, 2022 08:16
@sorpaas
Copy link
Member

sorpaas commented Aug 31, 2022

LGTM, but please fix the code style issue complained by the CI.

@tgmichel tgmichel requested a review from sorpaas September 6, 2022 14:50
@sorpaas sorpaas merged commit 06eb4f2 into polkadot-evm:master Sep 8, 2022
abhijeetbhagat pushed a commit to web3labs/frontier that referenced this pull request Jan 11, 2023
* Handle `earliest` tag for `BlockNumber`

* Add ts-test

* editorconfig

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants