Skip to content

Disable HK swap temporarily#2536

Merged
sam0x17 merged 3 commits intomainfrom
disable-hk-swap
Mar 24, 2026
Merged

Disable HK swap temporarily#2536
sam0x17 merged 3 commits intomainfrom
disable-hk-swap

Conversation

@l0r1s
Copy link
Collaborator

@l0r1s l0r1s commented Mar 24, 2026

Disable HK swap temporarily because of an accounting error in the current implementation

@github-actions github-actions bot added the hotfix This PR needs to be merged very quickly and will likely skip testing on devnet and testnet label Mar 24, 2026
@github-actions
Copy link
Contributor

🚨🚨🚨 HOTFIX DETECTED 🚨🚨🚨

It looks like you are trying to merge a hotfix PR into main. If this isn't what you wanted to do, and you just wanted to make a regular PR, please close this PR, base your changes off the devnet-ready branch and open a new PR into devnet ready.

If you are trying to merge a hotfix PR, please complete the following essential steps:

  1. go ahead and get this PR into main merged, so we can get the change in as quickly as possible!
  2. merge main into testnet, bumping spec_version
  3. deploy testnet
  4. merge testnet into devnet, bumping spec_version
  5. deploy devnet
  6. merge devnet into devnet-ready

If you do not complete these steps, your hotfix may be inadvertently removed in the future when branches are promoted to main, so it is essential that you do so.

#[pallet::weight(Weight::from_parts(161_700_000, 0)
.saturating_add(T::DbWeight::get().reads(17_u64))
.saturating_add(T::DbWeight::get().writes(10_u64)))]
pub fn swap_coldkey(
Copy link
Collaborator

Choose a reason for hiding this comment

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

don't do it for swap_coldkey?

@l0r1s l0r1s force-pushed the disable-hk-swap branch from b33c621 to b04bf14 Compare March 24, 2026 22:06
@sam0x17 sam0x17 merged commit 036cfe0 into main Mar 24, 2026
23 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hotfix This PR needs to be merged very quickly and will likely skip testing on devnet and testnet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants