zeitgeist
zeitgeist copied to clipboard
Remove `MarketStatus::Suspended`
This status is unused and should be removed completely.
Note that removing an item from the MarketStatus
enum will require a storage migration. This is what the substrate docs say about the encoding of enums:
A fixed number of variants, each mutually exclusive and potentially implying a further value or series of values. Encoded as the first byte identifying the index of the variant that the value is. Any further bytes are used to encode any data that the variant implies. Thus, no more than 256 variants are supported.
So...
#[derive(Clone, Copy, Decode, Encode, Eq, MaxEncodedLen, PartialEq, RuntimeDebug, TypeInfo)]
pub enum MarketStatus {
Proposed,
Active,
Suspended,
// All of these need to be migrated:
Closed,
CollectingSubsidy,
InsufficientSubsidy,
Reported,
Disputed,
Resolved,
}