Skip to content
Snippets Groups Projects

feat: Identity removal reason

Merged Pascal Engélibert requested to merge tuxmain/idty-rm-reason into master
All threads resolved!

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • changed milestone to %runtime-700

    • Resolved by Hugo Trentesaux

      Si je résume on a :

      enum IdtyRemovalReason {
          Expired, // une identité expire si elle n'a pas été renouvelée à temps
          Manual,
          Revoked, // uniquement par certificat de révocation
          Other(Other),
      }

      et

      enum IdtyRemovalWotReason {
          LostCert, // member passé sous le seuil de certifications dans la toile principale
          MembershipExpired, // adhésion non renouvelée
          Other,
      }
      • Manual n'est pas utilisé (et devrait l'être dans certains tests)
      • Other(Other) ne correspond dans notre cas qu'à des actions de type web of trust, on pourrait l'appeler Wot(Reason), de toutes façons les pallets sont couplées, ou même External(Reason)
      • Other je ne comprends pas dans quel cas le retrait de l'identité est dû à la toile de confiance mais ce n'est ni une certification perdue, ni une expiration d'adhésion

      D'ailleurs en relisant ça, je me rends compte que la logique de retrait des identités en cas de perte de certification n'est toujours pas claire (même si c'est moi qui l'ai fait dans !166 (merged)). Il faut que je revoie ça et ajoute des tests.

  • added 5 commits

    Compare with previous version

  • requested review from @HugoTrentesaux

  • Ce n'est pas encore ajouté à la CI, mais il faut essayer de garder des benchmarks qui compilent (cargo check --features runtime-benchmarks).

  • added 2 commits

    • ba57fad7 - Fix benchmarks
    • cadb0753 - Rename associated type Other to OtherReason

    Compare with previous version

  • Hugo Trentesaux resolved all threads

    resolved all threads

  • Hugo Trentesaux approved this merge request

    approved this merge request

  • Pascal Engélibert enabled an automatic merge when the pipeline for cadb0753 succeeds

    enabled an automatic merge when the pipeline for cadb0753 succeeds

  • Pascal Engélibert canceled the automatic merge

    canceled the automatic merge

  • mentioned in commit 71bc932e

  • Cédric Moreau changed milestone to %runtime-600

    changed milestone to %runtime-600

  • Cédric Moreau changed milestone to %runtime-700

    changed milestone to %runtime-700

  • Please register or sign in to reply
    Loading