Resolve "Identity pallet events are duplicated"

Closes #115 (closed)

changed milestone to %runtime-401

added C-binary label

assigned to @c-geek

marked this merge request as ready

added 3 commits

• 58eed892 - fix(!115 (merged)): remove IdtyEvent type
• 23e08b5f - fix(!115 (merged)): use value rather than references
• 2219a948 - fix(!115 (merged)): remove the `_ => {}` matching pattern

Compare with previous version

added RN-runtime label

added 1 commit

• d8b8ef19 - fix(!115 (merged)): fmt & clippy

Compare with previous version

requested review from @HugoTrentesaux

Maybe @tuxmain you could review also?

Pour moi les deux types d'événements n'ont pas le même rôle : l'un est un événement Substrate et l'autre est uniquement métier. Fusionner les deux oblige à dupliquer des données (par exemple les IdtyIndex redondants avec certaines variantes) et à gérer le __Ignore qui n'a rien à faire dans la sémantique des identités. En évitant le couplage on doit faire quelques conversions mais on choisit plus finement les champs.

Je vous laisse merger si vous voulez et à part ça tout est OK, mais personnellement je ne vois pas trop l'intérêt.

OK, à partir du moment où les "événements Substrate" comme tu les appelles ne sont effectivement pas exploités pour de la logique métier, alors oui nous sommes obligés de créer des événements dédiés et métier pour ça.

Je comprends, ça me va, je suis même plutôt contre ma propre MR du coup

closed

mentioned in issue #115 (closed)