Remove "owner_key" in identity confirmation event

It could be useful to include the identity index of the newly created identity in the identity.created event.

The identity confirmation event does not really need the owner key.

added S-needdiscussion label

We could also remove "owner_key" in identity confirmation event.

It could be useful to include the identity index of the newly created identity in the identity.created event.

Already there:

pub enum Event<T: Config> {
    /// A new identity has been created.
    IdtyCreated {
        idty_index: T::IdtyIndex,
        owner_key: T::AccountId,
    },
    /// An identity has been confirmed by its owner.
    IdtyConfirmed {
        idty_index: T::IdtyIndex,
        owner_key: T::AccountId,
        name: IdtyName,
    },
    /// An identity has been validated.
    IdtyValidated { idty_index: T::IdtyIndex },
    IdtyChangedOwnerKey {
        idty_index: T::IdtyIndex,
        new_owner_key: T::AccountId,
    },
    /// An identity has been revoked.
    IdtyRevoked {
        idty_index: T::IdtyIndex,
        reason: RevocationReason,
    },
    /// An identity has been removed.
    IdtyRemoved {
        idty_index: T::IdtyIndex,
        reason: RemovalReason,
    },
}

remove "owner_key" in identity confirmation event.

ok

changed title from Add index of newly created identity in creation event to Remove "owner_key" in identity confirmation event

changed the description

removed S-needdiscussion label

assigned to @tuxmain

added S-doing label

mentioned in merge request !311 (merged)

closed with merge request !311 (merged)