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

feat: custom upgrade module #17

Merged
merged 5 commits into from
Jan 6, 2023
Merged

feat: custom upgrade module #17

merged 5 commits into from
Jan 6, 2023

Conversation

j75689
Copy link
Contributor

@j75689 j75689 commented Dec 30, 2022

Description

Customize the upgrade module for BNB Inscription, and delete useless APIs from the original module.

Rationale

refer to: bnb-chain/greenfield-cosmos-sdk#82

Example

refer to: bnb-chain/greenfield-cosmos-sdk#82

Changes

Notable changes:

  • upgrade module

@j75689 j75689 force-pushed the upgrade_module branch 2 times, most recently from 716fa02 to ccef85b Compare January 3, 2023 10:11
@j75689 j75689 marked this pull request as ready for review January 3, 2023 10:30
app/app.go Outdated
authtypes.NewModuleAddress(govtypes.ModuleName).String(),
)
// Register the upgrade keeper
upgradeHandler := map[string]upgradetypes.UpgradeHandler{
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is not a suitable place to define the detailed handler.

@unclezoro unclezoro merged commit 958298a into develop Jan 6, 2023
j75689 added a commit that referenced this pull request Jan 6, 2023
* disable unnecessary modules (#14)

* feat: pass config to app when creating new app (#16)

* feat: pass config to app when create new app

* remove unused code

* deployment: fix relayer key generation (#19)

* feat: init balance of relayers in genesis state (#21)

* ci: fix release flow (#20)

* feat: custom upgrade module (#17)

* release: prepare for v0.0.2 (#22)

Co-authored-by: KeefeL <[email protected]>
Co-authored-by: yutianwu <[email protected]>
@unclezoro unclezoro deleted the upgrade_module branch April 18, 2023 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants