Feat/identity changes (#14)

* Initial changes

* Fixed errors for sub

* Removed .github folder

frame/identity/src/lib.rs

@@ -264,10 +264,10 @@ impl<
 #[repr(u64)]
 #[derive(Encode, Decode, Clone, Copy, PartialEq, Eq, BitFlags, RuntimeDebug)]
 pub enum IdentityField {
-	Display        = 0b0000000000000000000000000000000000000000000000000000000000000001,
+	// Display        = 0b0000000000000000000000000000000000000000000000000000000000000001,
 	Legal          = 0b0000000000000000000000000000000000000000000000000000000000000010,
 	Web            = 0b0000000000000000000000000000000000000000000000000000000000000100,
-	Riot           = 0b0000000000000000000000000000000000000000000000000000000000001000,
+	Discord        = 0b0000000000000000000000000000000000000000000000000000000000001000,
 	Email          = 0b0000000000000000000000000000000000000000000000000000000000010000,
 	PgpFingerprint = 0b0000000000000000000000000000000000000000000000000000000000100000,
 	Image          = 0b0000000000000000000000000000000000000000000000000000000001000000,
@@ -307,7 +307,7 @@ pub struct IdentityInfo {
 	/// reasonable context.
 	///
 	/// Stored as UTF-8.
-	pub display: Data,
+	// pub display: Data,
 
 	/// The full legal name in the local jurisdiction of the entity. This might be a bit
 	/// long-winded.
@@ -322,10 +322,10 @@ pub struct IdentityInfo {
 	/// Stored as UTF-8.
 	pub web: Data,
 
-	/// The Riot/Matrix handle held by the controller of the account.
+	/// The Discord handle held by the controller of the account.
 	///
 	/// Stored as UTF-8.
-	pub riot: Data,
+	pub discord: Data,
 
 	/// The email address of the controller of the account.
 	///
@@ -488,7 +488,9 @@ decl_error! {
 		/// Sender is not a sub-account.
 		NotSub,
 		/// Sub-account isn't owned by sender.
-		NotOwned
+		NotOwned,
+		/// Sub-accounts not enabled
+		SubNotEnabled
 	}
 }
 
@@ -642,6 +644,7 @@ decl_module! {
 			.saturating_add(T::WeightInfo::set_subs_new(subs.len() as u32)) // S: Assume all subs are new.
 		]
 		fn set_subs(origin, subs: Vec<(T::AccountId, Data)>) -> DispatchResultWithPostInfo {
+			ensure!(false, Error::<T>::SubNotEnabled);
 			let sender = ensure_signed(origin)?;
 			ensure!(<IdentityOf<T>>::contains_key(&sender), Error::<T>::NotFound);
 			ensure!(subs.len() <= T::MaxSubAccounts::get() as usize, Error::<T>::TooManySubAccounts);
@@ -1041,6 +1044,7 @@ decl_module! {
 		/// sub identity of `sub`.
 		#[weight = T::WeightInfo::add_sub(T::MaxSubAccounts::get())]
 		fn add_sub(origin, sub: <T::Lookup as StaticLookup>::Source, data: Data) -> DispatchResult {
+			ensure!(false, Error::<T>::SubNotEnabled);
 			let sender = ensure_signed(origin)?;
 			let sub = T::Lookup::lookup(sub)?;
 			ensure!(IdentityOf::<T>::contains_key(&sender), Error::<T>::NoIdentity);
@@ -1069,6 +1073,7 @@ decl_module! {
 		/// sub identity of `sub`.
 		#[weight = T::WeightInfo::rename_sub(T::MaxSubAccounts::get())]
 		fn rename_sub(origin, sub: <T::Lookup as StaticLookup>::Source, data: Data) {
+			ensure!(false, Error::<T>::SubNotEnabled);
 			let sender = ensure_signed(origin)?;
 			let sub = T::Lookup::lookup(sub)?;
 			ensure!(IdentityOf::<T>::contains_key(&sender), Error::<T>::NoIdentity);
@@ -1085,6 +1090,7 @@ decl_module! {
 		/// sub identity of `sub`.
 		#[weight = T::WeightInfo::remove_sub(T::MaxSubAccounts::get())]
 		fn remove_sub(origin, sub: <T::Lookup as StaticLookup>::Source) {
+			ensure!(false, Error::<T>::SubNotEnabled);
 			let sender = ensure_signed(origin)?;
 			ensure!(IdentityOf::<T>::contains_key(&sender), Error::<T>::NoIdentity);
 			let sub = T::Lookup::lookup(sub)?;
@@ -1112,6 +1118,7 @@ decl_module! {
 		/// controller of an account is maliciously registered as a sub-account.
 		#[weight = T::WeightInfo::quit_sub(T::MaxSubAccounts::get())]
 		fn quit_sub(origin) {
+			ensure!(false, Error::<T>::SubNotEnabled);
 			let sender = ensure_signed(origin)?;
 			let (sup, _) = SuperOf::<T>::take(&sender).ok_or(Error::<T>::NotSub)?;
 			SubsOf::<T>::mutate(&sup, |(ref mut subs_deposit, ref mut sub_ids)| {