-
Notifications
You must be signed in to change notification settings - Fork 3.6k
feat(uniond): v1.2.0 migration #4781
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(uniond): v1.2.0 migration #4781
Conversation
PoisonPhang
commented
Jul 8, 2025
- migration for v1.2
- Mints token
- Changes default token in all contexts
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
2fc03ae
to
be8b5c8
Compare
Docs 🤌✨ Deployment complete! Take a peek over at https://4e216de6.docs-ajn.pages.dev |
💞💞💞💞💞💞💞💞💞💞💞💞💞 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good, let's have multiple eyes review this and prepare a testnet upgrade. BTW, for UNION_FOUNDATION_MULTI_SIG lets switch on the chain ID for the upgrade such that we send the supply on the whale account in testnet, and the foundation multisig in mainent
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a few questions and nits
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR implements the v1.2.0 migration for uniond which introduces a new U token and removes POA (Proof of Authority) functionality. The migration mints the total supply of U tokens, undelegates existing delegations, burns old tokens, and updates all module parameters to use the new token denomination.
Key changes:
- Implements comprehensive v1.2.0 upgrade handler that mints U tokens and burns legacy tokens
- Removes POA module dependencies and references throughout the codebase
- Updates module parameters across staking, mint, gov, crisis, distribution, and feemarket modules to use new U token
Reviewed Changes
Copilot reviewed 16 out of 18 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
uniond/app/upgrades/v1_2_0/upgrade.go | Core migration logic for token minting, delegation management, and parameter updates |
uniond/app/upgrades/v1_2_0/constants.go | Migration constants including POA store deletion |
uniond/app/upgrades/types.go | Extended AppKeepers structure with additional required keepers |
uniond/app/upgrades.go | Registered v1.2.0 upgrade and populated keeper dependencies |
uniond/app/post.go | Added StakingKeeper dependency to post handler |
uniond/app/wasm.go | Added StakingKeeper to post handler options |
versions/versions.json | Updated version lists for networks |
uniond/go.mod | Updated dependencies and moved grpc-gateway to direct imports |
uniond/proto.nix | Removed POA proto dependencies |
uniond/docs/static/openapi.yml | Removed POA API endpoints and updated validator schema |