-
Notifications
You must be signed in to change notification settings - Fork 165
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
Resuming the network #631
Resuming the network #631
Conversation
Contrary to description this PR does not seem to remove the schedule cold key swap extrisinc |
@ppolewicz we kept it under a test cfg gate , otherwise removing it would have broken all our tests : https://github.com/opentensor/subtensor/pull/631/files#diff-9d85ff26db441ec381d86188b389b226449eee4e9847bf0ce3301c0bb6cf93a0R2107. I have tested it locally, and this extrinsic no longer appears in the runtime We will remove excise it completely at a later date |
@distributedstatemachine maybe rather than delete it we might keep it in the source code under that cfg flag in case a major validator or subnet coldkey ever gets compromised again? |
Description
This PR
Related Issue(s)
Type of Change
Breaking Change
If this PR introduces a breaking change, please provide a detailed description of the impact and the migration path for existing applications.
Checklist
cargo fmt
andcargo clippy
to ensure my code is formatted and linted correctlyScreenshots (if applicable)
Please include any relevant screenshots or GIFs that demonstrate the changes made.
Additional Notes
Please provide any additional information or context that may be helpful for reviewers.