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

remove locked coin staking #9046

Merged
merged 1 commit into from
Mar 11, 2023
Merged

Conversation

emmazzz
Copy link
Contributor

@emmazzz emmazzz commented Mar 9, 2023

Description

This PR removes support for locked coin staking in various places.

Test Plan

cargo test


If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.

Type of Change (Check all that apply)

  • user-visible impact
  • breaking change for a client SDKs
  • breaking change for FNs (FN binary must upgrade)
  • breaking change for validators or node operators (must upgrade binaries)
  • breaking change for on-chain data layout
  • necessitate either a data wipe or data migration

Release notes

@vercel
Copy link

vercel bot commented Mar 9, 2023

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

Name Status Preview Comments Updated
explorer ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 11, 2023 at 6:59PM (UTC)
explorer-storybook ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 11, 2023 at 6:59PM (UTC)
wallet-adapter ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 11, 2023 at 6:59PM (UTC)

@vercel vercel bot temporarily deployed to Preview – wallet-adapter March 9, 2023 07:16 Inactive
@vercel vercel bot temporarily deployed to Preview – explorer-storybook March 9, 2023 07:17 Inactive
@emmazzz emmazzz force-pushed the remove_locked_staking branch from 70cb3c4 to 4fe7d8d Compare March 9, 2023 07:42
@vercel vercel bot temporarily deployed to Preview – explorer-storybook March 9, 2023 07:43 Inactive
@vercel vercel bot temporarily deployed to Preview – wallet-adapter March 9, 2023 07:44 Inactive
@emmazzz emmazzz force-pushed the remove_locked_staking branch from 4fe7d8d to 8564d99 Compare March 11, 2023 00:38
@vercel vercel bot temporarily deployed to Preview – explorer-storybook March 11, 2023 00:39 Inactive
@vercel vercel bot temporarily deployed to Preview – wallet-adapter March 11, 2023 00:40 Inactive
@emmazzz emmazzz force-pushed the remove_locked_staking branch from 8564d99 to 5c26ef9 Compare March 11, 2023 00:44
@vercel vercel bot temporarily deployed to Preview – wallet-adapter March 11, 2023 00:45 Inactive
@vercel vercel bot temporarily deployed to Preview – explorer-storybook March 11, 2023 00:45 Inactive
@emmazzz emmazzz force-pushed the remove_locked_staking branch from 5c26ef9 to 7fed825 Compare March 11, 2023 00:53
@vercel vercel bot temporarily deployed to Preview – wallet-adapter March 11, 2023 00:53 Inactive
@vercel vercel bot temporarily deployed to Preview – explorer-storybook March 11, 2023 00:54 Inactive
Copy link
Contributor

@patrickkuo patrickkuo left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@tzakian tzakian left a comment

Choose a reason for hiding this comment

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

Looks great to me!

@emmazzz emmazzz force-pushed the remove_locked_staking branch from 7fed825 to 3e43450 Compare March 11, 2023 18:58
@emmazzz emmazzz enabled auto-merge (squash) March 11, 2023 18:58
@vercel vercel bot temporarily deployed to Preview – wallet-adapter March 11, 2023 18:58 Inactive
@vercel vercel bot temporarily deployed to Preview – explorer-storybook March 11, 2023 18:58 Inactive
@emmazzz emmazzz merged commit 79c2165 into MystenLabs:main Mar 11, 2023
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