Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

revamp locking & reserving #13122

Closed
wants to merge 4 commits into from
Closed

Conversation

sam0x17
Copy link
Contributor

@sam0x17 sam0x17 commented Jan 10, 2023

Description

This PR is our initial stab at paritytech/polkadot-sdk#236, which involves revisiting the definitions of free, locked, and reserved, especially as these concepts relate to voting, aucitions/crowdloans, and staking. In particular reserving will be replaced with "holding" via a type-safe HoldReason, and

This will entail:

  • generic #[pallet::composite] syntax for definining composite enums that can then be used at the runtime level
  • HoldReason / type-safe reserve/hold identifier via #[pallet::composite]
  • FreezeReason / type-safe lock/freeze identifier via #[pallet::composite]
  • TBD, see original issue Tokens: Revamp locking & reserving polkadot-sdk#236

Related Issued

@sam0x17 sam0x17 added B7-runtimenoteworthy C3-medium PR touches the given topic and has a medium impact on builders. D9-needsaudit 👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited labels Jan 10, 2023
@sam0x17 sam0x17 self-assigned this Jan 10, 2023
@github-actions github-actions bot added the A3-in_progress Pull request is in progress. No review needed at this stage. label Jan 10, 2023
@sam0x17 sam0x17 force-pushed the sam-revamp-locking-and-reserving branch from 8e3081b to f34ce85 Compare January 18, 2023 23:02
@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2284472

@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-each-crate
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2284487

@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2284473

@sam0x17 sam0x17 closed this Jan 27, 2023
@louismerlin louismerlin removed the D9-needsaudit 👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited label May 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A3-in_progress Pull request is in progress. No review needed at this stage. C3-medium PR touches the given topic and has a medium impact on builders.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants