feat: dynamically fetch entropy gas limits from blockchain #763
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
- Add EntropyGasLimitTable component to fetch gas limits using getProviderInfoV2 - Update contract addresses page to display dynamic gas limits - Remove static gas limit column from EntropyDeploymentTable - Add IEntropyV2.json ABI for blockchain queries - Follow same pattern as EntropyFeeTable for async data fetching Co-Authored-By: Jayant <[email protected]>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
- Fix pre-commit CI failure by applying prettier formatting - Reformat promise chain and simplify ternary operator Co-Authored-By: Jayant <[email protected]>
- Remove abis/IEntropyV2.json file as requested - Delete separate EntropyGasLimitTable component - Integrate gas limit column back into EntropyDeploymentTable - Update contract-addresses.mdx to use unified table approach - Currently showing static gas limits, dynamic fetching to be implemented next Co-Authored-By: Jayant <[email protected]>
- Add inline ABI for getProviderInfoV2 method to avoid abis folder dependency - Integrate dynamic gas limit fetching directly into EntropyDeploymentTable - Maintain fallback to static values when blockchain queries fail - Show 'Loading...' state while fetching dynamic values from contracts - Successfully combines contract addresses and gas limits in single table Co-Authored-By: Jayant <[email protected]>
- Fix formatting issues identified by pre-commit prettier hook - Ensure code style consistency across the repository Co-Authored-By: Jayant <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR updates the Entropy contract addresses documentation page to dynamically fetch gas limits from the blockchain instead of displaying static values. The implementation follows the same pattern as the existing
EntropyFeeTablecomponent, using thegetProviderInfoV2method on the Entropy contract to retrieve real-time gas limit data.Type of Change
Areas Affected
pages/entropy/contract-addresses.mdx- Updated to use new dynamic gas limit componentcomponents/EntropyGasLimitTable.tsx- New component for fetching gas limits from blockchaincomponents/EntropyDeploymentTable.tsx- Removed static gas limit columnabis/IEntropyV2.json- Added ABI file for blockchain queriesKey Changes
New Dynamic Component: Created
EntropyGasLimitTablethat:getProviderInfoV2contract methodEntropyFeeTableUpdated Page Structure: Modified contract addresses page to:
Removed Static Data: Eliminated static gas limit column from deployment table since values are now fetched dynamically
Critical Review Areas
IEntropyV2.jsonis the correct/latest version compatible with deployed contractsChecklist
pre-commit run --all-filesto check for linting errorsTesting
localhost:3000/entropy/contract-addressesScreenshots
The screenshot shows the updated contract addresses page with dynamic gas limit tables for both mainnet and testnet networks, displaying real values fetched from the blockchain.
Related Issues
Addresses request from Jayant ([email protected]) to update entropy documentation to fetch gas limits dynamically from blockchain instead of using static values.
Additional Notes
EntropyFeeTablefor consistencydefaultGasLimitfield of theProviderInfostruct returned bygetProviderInfoV2Contributor Information