Closed m-Peter closed 3 months ago
[!WARNING]
Rate limit exceeded
@sideninja has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 8 minutes and 57 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between ae59b67ac5ca198f9c0cfa1e88834577304dfb6c and df9a09907f7cd66a4e8c0b3e5724cce7b718c74d.
The recent changes enhance the BlockChainAPI
and WalletAPI
by simplifying the handling of block number and hash parameters while ensuring robust default behavior. The Requester
interface has been updated to support gas estimation at specific EVM block heights, improving functionality. Overall, these modifications align the API with standards and enhance its usability.
Files | Change Summary |
---|---|
api/api.go , api/wallet.go |
Changed blockNumberOrHash from pointer to value in several methods, introduced default value handling. |
services/requester/requester.go |
Updated EstimateGas method to include an evmHeight parameter for improved gas estimation accuracy. |
tests/web3js/eth_deploy_contract_and_interact_test.js |
Added tests for gas estimation in contract deployment and interaction under different block conditions. |
Objective | Addressed | Explanation |
---|---|---|
Match behavior regarding JSON-RPC APIs with default block parameter (#398) | ā | |
Handle custom block numbers in eth_estimateGas endpoint (#380) |
ā |
š° In the blockchain woods so wide,
A hop of change, we take in stride.
With gas and blocks in harmony,
The API dances, wild and free!
No more confusion, just clear paths,
A rabbit's joy in code it hath! šæāØ
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Closes: https://github.com/onflow/flow-evm-gateway/issues/398 Closes: https://github.com/onflow/flow-evm-gateway/issues/380
Description
rpc.BlockNumberOrHash
required, for the respective JSON-RPC APIseth_estimateGas
to accept arpc.BlockNumberOrHash
parameterFor contributor use:
master
branchFiles changed
in the Github PR explorerSummary by CodeRabbit
New Features
Bug Fixes
Tests
Refactor