Skip to content

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

Merged
merged 21 commits into from
Jul 16, 2025

Conversation

PoisonPhang
Copy link
Member

  • migration for v1.2
  • Mints token
  • Changes default token in all contexts

Copy link

vercel bot commented Jul 8, 2025

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
site ⬜️ Ignored (Inspect) Visit Preview Jul 16, 2025 4:52pm

@PoisonPhang PoisonPhang force-pushed the prod-415-write-migration-to-mint-u-and-remove-poa branch from 2fc03ae to be8b5c8 Compare July 8, 2025 20:02
Copy link

github-actions bot commented Jul 8, 2025

Docs 🤌

✨ Deployment complete! Take a peek over at https://4e216de6.docs-ajn.pages.dev
✨ Deployment alias URL: https://head.docs-ajn.pages.dev


@demiGOD-web
Copy link

💞💞💞💞💞💞💞💞💞💞💞💞💞
Keep building sers💪🏾

Copy link
Contributor

@hussein-aitlahcen hussein-aitlahcen left a 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

@PoisonPhang PoisonPhang marked this pull request as ready for review July 15, 2025 20:52
@PoisonPhang PoisonPhang requested a review from cor July 15, 2025 20:52
Copy link
Contributor

@benluelo benluelo left a 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

@PoisonPhang PoisonPhang requested a review from benluelo July 15, 2025 22:52
@cor cor requested a review from Copilot July 15, 2025 23:27
Copy link

@Copilot Copilot AI left a 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

@PoisonPhang PoisonPhang deployed to docs-preview July 16, 2025 16:52 — with GitHub Actions Active
@PoisonPhang PoisonPhang requested a review from benluelo July 16, 2025 16:52
@PoisonPhang PoisonPhang merged commit e23236f into main Jul 16, 2025
119 checks passed
@PoisonPhang PoisonPhang deleted the prod-415-write-migration-to-mint-u-and-remove-poa branch July 16, 2025 19:25
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.

6 participants