Skip to content

Conversation

@nidhi-singh02
Copy link
Contributor

@nidhi-singh02 nidhi-singh02 commented Aug 14, 2025

Summary

In requestWithCallback comment, there's a mention of requestWithCallbackAndGasLimit in case more gas is required.

Updating the comment so it points to the Entropyv2 interface request method.

Rationale

requestWithCallbackAndGasLimit does not exist, redirect the user to use the version 2 in case they want to set their custom gas limit

How has this been tested?

  • Current tests cover my changes
  • Added new tests
  • Manually tested the code

@nidhi-singh02 nidhi-singh02 self-assigned this Aug 14, 2025
@vercel
Copy link

vercel bot commented Aug 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Project Deployment Preview Comments Updated (UTC)
api-reference Ready Preview Comment Aug 14, 2025 9:52am
6 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
component-library ⬜️ Skipped Aug 14, 2025 9:52am
developer-hub ⬜️ Skipped Aug 14, 2025 9:52am
entropy-explorer ⬜️ Skipped Aug 14, 2025 9:52am
insights ⬜️ Skipped Aug 14, 2025 9:52am
proposals ⬜️ Skipped Aug 14, 2025 9:52am
staking ⬜️ Skipped Aug 14, 2025 9:52am

@vercel vercel bot temporarily deployed to Preview – entropy-explorer August 14, 2025 09:48 Inactive
@vercel vercel bot temporarily deployed to Preview – proposals August 14, 2025 09:48 Inactive
@vercel vercel bot temporarily deployed to Preview – insights August 14, 2025 09:48 Inactive
@vercel vercel bot temporarily deployed to Preview – component-library August 14, 2025 09:48 Inactive
@vercel vercel bot temporarily deployed to Preview – staking August 14, 2025 09:48 Inactive
@vercel vercel bot temporarily deployed to Preview – developer-hub August 14, 2025 09:48 Inactive
@nidhi-singh02 nidhi-singh02 merged commit de1b47e into main Aug 14, 2025
10 checks passed
@nidhi-singh02 nidhi-singh02 deleted the ientropy-comments-fix branch August 14, 2025 13:58
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.

4 participants