Add status to idty
Merge request reports
Activity
assigned to @HugoTrentesaux
added 1 commit
- a2ed9653 - feat: add idtyValidated and idtyRemoved events
The 2 missings points are added. Ok to merge for me.
Edited by poka- Resolved by poka
Bon en fait non on ne peut pas supprimer les identités de la base de donnée comme ça ...
FATAL sqd:processor QueryFailedError: update or delete on table "identity" violates foreign key constraint "FK_70592e488b2e75cd8a2fa798261" on table "cert"
Il faudrait aussi supprimer les certifications qui y sont liés, mais on part sur des cascades de suppressions qui n'ont pas lieu d'être dans un indexer.
Je vais donc laisser ces identités et les passer en status
Removed
. Il faudrait un champ pour y ajouter la raison (Expired, Manual, Other, Revoked
).Edited by poka
- Resolved by Hugo Trentesaux
Voilà ce que ça donne, avec le champ reason nullable:
Je ne sais pas ce que tu en penses. j'ai songé à indiquer la raison directement en status, mais pour les raisons Manual et Other ça ne serait pas assez explicite (je ne sais pas à quelle réalité ces raisons correspondent en blockchain).
Ca permet de garder la trace de l'idtyIndex, et ainsi suivre l'historique des évolutions de status avec ce genre de requête:
query { events(where: {pallet_eq: "Identity", args_jsonContains: "{\"idtyIndex\": 14156}"}) { id name pallet args block { height } } }
Edited by poka
mentioned in issue #1 (closed)
- Resolved by poka
Qu'est-ce que tu utilises comme formatteur pour le schéma graphql ? Il faudrait que j'utilise le même pour éviter d'avoir des changements d'indentation à chaque commit et faciliter la lecture.
- Resolved by Hugo Trentesaux
assigned to @pokapow and unassigned @HugoTrentesaux
requested review from @HugoTrentesaux