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

add UnlockAllVestingAccounts func to upgrade handler #171

Merged
merged 2 commits into from
Apr 11, 2022

Conversation

catShaark
Copy link
Contributor

@catShaark catShaark commented Mar 31, 2022

Impl proposal 7 to unlock all vesting account on chain
Logic for unlocking is in upgrade handler, once dig upgrade to v2, all vesting accounts will be unlocked.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@faddat
Copy link
Collaborator

faddat commented Apr 11, 2022

@catShaark

@nguyenvuong1122000

hey guys is this landed in the code?

We've definitely got to get this in.

Copy link
Collaborator

@faddat faddat left a comment

Choose a reason for hiding this comment

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

looks good but I am going to defer to @nguyenvuong1122000 for final approval of this code.

@vuong177
Copy link
Contributor

I think it's OK

@vuong177 vuong177 merged commit 7c4c52c into master Apr 11, 2022
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.

3 participants