Closed zakir-code closed 5 days ago
This pull request encompasses a series of changes across multiple files, focusing primarily on the ERC20 token handling within the application. Key modifications include the replacement of the TokenPair
type with ERC20Token
, adjustments to various structures and methods, simplification of configuration parameters, and the introduction of new message types. Additionally, several files related to deprecated functionalities have been removed, and the Makefile
has been updated to reflect changes in linting rules. Overall, the changes aim to streamline the ERC20 module and improve its integration within the broader application framework.
File Path | Change Summary |
---|---|
Makefile | Updated check-no-lint target to reduce allowed nolint comments from 29 to 26. |
api/fx/erc20/v1/genesis.pulsar.go | Replaced TokenPair with ERC20Token , updated GenesisState and Params structs, removing certain fields. |
api/fx/erc20/v1/types.pulsar.go | Introduced new message types: ERC20Token , BridgeToken , IBCToken , and an enum Owner . |
app/app_test.go | Added new message types to the deprecated map in Test_MsgServiceRouter . |
app/encoding_test.go | Adjusted assertion in TestRegisterInterfaces from 298 to 293 . |
app/genesis.go | Updated NewDefAppGenesisByDenom function for DenomMetadata population. |
app/genesis.json | Removed enable_evm_hook and ibc_timeout parameters from erc20 section. |
app/interface_registry.json | Removed several entries related to fx.erc20 module from TypeURLMap . |
app/keepers/keepers.go | Refactored NewAppKeeper function and updated initialization of keeper instances. |
docs/swagger-ui/swagger.yaml | Removed several API endpoints and restructured response schemas for ERC20-related queries. |
go.mod | Updated dependencies, reintroducing golang.org/x/exp and replacing some dependencies with forked versions. |
proto/fx/erc20/v1/erc20.proto | Deleted file defining Protocol Buffers schema for ERC20 functionality. |
proto/fx/erc20/v1/genesis.proto | Updated import statement and changed GenesisState message to use ERC20Token . |
proto/fx/erc20/v1/legacy.proto | Introduced new message definitions for ERC20 conversions, marked as deprecated. |
proto/fx/erc20/v1/query.proto | Updated import statement, marked several RPC methods as deprecated, and changed return types. |
proto/fx/erc20/v1/types.proto | Introduced new Protocol Buffers file defining message types related to ERC20 tokens. |
tests/crosschain_suite.go | Removed SelectTokenMetadata method. |
tests/crosschain_test.go | Overhauled CrossChainTest function for structured testing of cross-chain interactions. |
tests/erc20_suite.go | Removed several methods related to ERC20 token handling. |
x/crosschain/keeper/bridge_call_in.go | Renamed variables for clarity and altered logic for handling bridge tokens. |
x/crosschain/keeper/bridge_call_out.go | Updated handling of outgoing bridge calls with new parameters. |
x/erc20/keeper/keeper.go | Significant restructuring of Keeper struct, including new fields and methods related to ERC20 token management. |
x/erc20/keeper/msg_server.go | Transitioned from Keeper method receiver to a new msgServer type, altering method signatures and logic. |
x/erc20/module.go | Removed command methods from AppModuleBasic , updated service registration, and added error handling in InitGenesis and ExportGenesis . |
x/erc20/types/errors.go | Updated error messages for ErrUndefinedOwner and ErrERC20TokenPairDisabled . |
x/erc20/types/events.go | Removed several constants related to ERC20 events and attributes. |
Makefile
and bridge functionality may connect to cross-chain operations.🐇 In the meadow, we hop with glee,
ERC20 tokens, as bright as can be!
With bridges and tokens, we dance and play,
Simplified paths light up our way.
Hopping through changes, we cheer and sing,
A world of tokens, oh what joy they bring! 🌼
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Summary by CodeRabbit
Release Notes
New Features
ERC20Token
,BridgeToken
, andIBCToken
.Bug Fixes
Documentation
Chores
Tests