Skip to content

Conversation

@jayantk
Copy link
Contributor

@jayantk jayantk commented May 15, 2025

Summary

Remove chains that are no longer live & their corresponding entropy deployments

How has this been tested?

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

@vercel
Copy link

vercel bot commented May 15, 2025

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

Name Status Preview Comments Updated (UTC)
api-reference ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 15, 2025 5:03pm
component-library ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 15, 2025 5:03pm
entropy-debugger ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 15, 2025 5:03pm
entropy-explorer ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 15, 2025 5:03pm
insights ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 15, 2025 5:03pm
proposals ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 15, 2025 5:03pm
staking ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 15, 2025 5:03pm

const gas = Math.trunc((await deployTx.estimateGas()) * gasMultiplier);
const gasPrice = Math.trunc(
Number(await this.getGasPrice()) * gasPriceMultiplier,
);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

drive-by to support non-integer multipliers

@jayantk jayantk merged commit 7aebe7a into main May 15, 2025
10 checks passed
@jayantk jayantk deleted the entropy_error20 branch May 15, 2025 17:04
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.

3 participants