feat(smith-members): weird things

pallets/authority-members/src/lib.rs

@@ -530,6 +530,7 @@ impl<T: Config> pallet_session::SessionManager<T::ValidatorId> for Pallet<T> {
         let members_ids_to_add = IncomingAuthorities::<T>::take();
         let members_ids_to_del = OutgoingAuthorities::<T>::take();
 
+        // TODO: what if we have outgoing members but no incoming ones?
         if members_ids_to_add.is_empty() {
             if members_ids_to_del.is_empty() {
                 // when no change to the set of autorities, return None

runtime/common/src/handlers.rs

@@ -95,6 +95,7 @@ impl<
                 // user will have to claim authority membership
             }
             sp_membership::Event::MembershipRemoved(idty_index) => {
+                // TODO: recursive call? (see authority-members -> revoke_membership)
                 let call = pallet_authority_members::Call::<Runtime>::remove_member {
                     member_id: *idty_index,
                 };