Skip to content

Conversation

@MudDev
Copy link

@MudDev MudDev commented Dec 7, 2025

This pull request refactors EpixChain's denomination constants and updates related configuration, genesis, and upgrade logic to use these new constants consistently across the codebase. It removes usage of test constants and centralizes chain-specific values, improving maintainability and reducing errors. Additionally, EVM and ERC20 module initialization and upgrade handlers are updated to properly set and initialize denom metadata and options.

Denomination and Metadata Refactoring

  • Added new constants for BaseDenom, DisplayDenom, Decimals, and WEPIXContract in evmd/constants.go, and replaced all previous references to test constants with these new values throughout the codebase.
  • Updated bank and mint genesis state creation to use the new denom constants for metadata, mint denom, and display denom. [1] [2]
  • Refactored the upgrade handler to update denom metadata and EVM params using the new constants, and ensured EVM coin info is initialized from bank metadata.

EVM and ERC20 Module Initialization

  • Set EvmDenom and ExtendedDenomOptions in EVM genesis state and upgrade handler using the new base denom constant. [1] [2]
  • Updated ERC20 genesis state to use the new token pair and contract address constants for the native token.

Testing and Configuration Updates

  • Removed usage of test constants in the test network setup, using the new denom constants for bond denom, min gas prices, and EVM chain ID extraction. [1] [2] [3]
  • Added a utility function to extract the EVM chain ID from the Cosmos chain ID for configuration.

Proposal Updates

  • Updated the v0.5.1 upgrade proposal JSON to reflect the new upgrade height and removed the expedited flag from the proposal.

@MudDev MudDev merged commit 1d6d1e9 into main Dec 7, 2025
11 of 28 checks passed
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