Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add block gas limit in each chain card #1511

Merged
merged 3 commits into from
Feb 4, 2025

Conversation

Setland34
Copy link
Contributor

@Setland34 Setland34 commented Jan 31, 2025

Related to #1426

Add block gas limit display to each chain card.

  • Add a new state variable blockGasLimit with a default value of 'Unknown'.
  • Fetch the block gas limit data for each chain using the chain's RPC URL.
  • Update the table in the chain card to include a new column for the block gas limit.
  • Display the fetched block gas limit in the new column, or 'Unknown' if the data is not available.

For more details, open the Copilot Workspace session.

Related to DefiLlama#1426

Add block gas limit display to each chain card.

* Add a new state variable `blockGasLimit` with a default value of 'Unknown'.
* Fetch the block gas limit data for each chain using the chain's RPC URL.
* Update the table in the chain card to include a new column for the block gas limit.
* Display the fetched block gas limit in the new column, or 'Unknown' if the data is not available.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/DefiLlama/chainlist/issues/1426?shareId=XXXX-XXXX-XXXX-XXXX).
Copy link

vercel bot commented Jan 31, 2025

@Setland34 is attempting to deploy a commit to the Defillama Team Team on Vercel.

A member of the Team first needs to authorize it.

Setland34 and others added 2 commits February 1, 2025 11:48
* Import Tooltip component
* Add a new table row for block gas limit with a tooltip to display the value
* Set the default value of block gas limit to 'Hover to see'
@mintdart mintdart merged commit 54f7dc4 into DefiLlama:main Feb 4, 2025
1 check failed
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