refac membership

Refac membership

closes #155 (closed) as it removes pending membership
closes #156 (closed) (not entirely, the rest will be done in a separate MR on smith wot)
closes #136 (closed) (it does not link instanciable pallets but fixes the test that were added to show the unwanted behavior)

• remove request_membership call
• remove PendingMembership storage item
• remove PendingMembershipsExpireOn storage item
• remove validate_identity call from identity pallet which is redundant with claim_membership of membership pallet
• improves IdtyStatus
• improves RevocationReason and RemovalReason
• split ConfirmPeriod ValidationPeriod AutorevocationPeriod DeletionPeriod
• remove useless IdtyEvent variants
• improves MembershipRemovalReason
• adds CertToUnconfirmed CertToRevoked variants for duniter-wot Error
• split DistanceEvaluationPending DistanceEvaluationNotRequested

This changes the way identity and membership pallet interact.

changed milestone to %runtime-800

added RN-runtime label

assigned to @HugoTrentesaux

added 2 commits

• 9e5e7df9 - add *Period
• 528ae661 - refac scheduling (tests ok)

Compare with previous version

added 3 commits

• 89b70eaa - refac
• 268c4293 - wip
• 874bdc68 - refac revocation and removal reason

Compare with previous version

added 4 commits

• 4c15b66a - remove removal other reason
• f5d83cdb - remove IdtyEvent change owner key
• efad6113 - cargo check all
• 984ca3fe - cargo check all tests

Compare with previous version

added 1 commit

• 568e2a29 - cargo test workspace quasi ok

Compare with previous version

added 1 commit

• 25dcbf41 - cargo test workspace ok

Compare with previous version

added 1 commit

• 86298b06 - cargo cucumber ok

Compare with previous version

changed the description

added 6 commits

• 0205e1a5 - check benchmarks ok
• 67491249 - more precise error messages
• e97b7b40 - test benchmark ok
• c39d14cc - refac identity creation checks
• b871e19d - refac certification creation checks
• 65772940 - add wot tests related to issue #136 (closed)

Compare with previous version

added 1 commit

• a420bb61 - clippy ‍♂

Compare with previous version

added 1 commit

• 056b85df - build live tests

Compare with previous version

added 3 commits

• 0dc746b4 - remove check_remove_identity
• 5bd4e5ad - add membership removal reasons
• a280bf4d - fix tests according to new MembershipRemovalReason

Compare with previous version

changed the description

marked the checklist item improves RevocationReason and RemovalReason as completed

marked the checklist item split ConfirmPeriod ValidationPeriod AutorevocationPeriod DeletionPeriod as completed

marked the checklist item remove useless IdtyEvent variants as completed

marked the checklist item improves MembershipRemovalReason as completed

marked the checklist item adds CertToUnconfirmed CertToRevoked variants for duniter-wot Error as completed

marked the checklist item split DistanceEvaluationPending DistanceEvaluationNotRequested as completed

requested review from @c-geek

added 4 commits

• 6b37235b - update doc
• 87ac3a6e - fix todo in test
• 2fe513c5 - remove force_remove_identity call
• f1f91e4a - remove remove_identity_consumers

Compare with previous version

added 2 commits

• 7b2a06f8 - add benchmark
• 245afe8b - update weightinfo and benchmarks accordingly

Compare with previous version

marked this merge request as ready

changed time estimate to 1w

closed

reopened

(oups)

mentioned in issue #137 (closed)

changed the description

C'est déjà plus clair sans le système de pending membership.

Il reste un fichier notes.txt : la MR n'est pas vraiment terminée ?

mentioned in issue #160 (closed)

mentioned in issue #161 (closed)

mentioned in issue #162 (closed)

added 4 commits

• 246f3811 - doc idtystatus in readme
• 879d8327 - update comment according to #159 (closed)
• 06ff3031 - update notes with links to issues
• 57edaab2 - delete notes.txt file

Compare with previous version

C'est déjà plus clair sans le système de pending membership.

Il reste un fichier notes.txt : la MR n'est pas vraiment terminée ?

approved this merge request

resolved all threads

merged

mentioned in commit 6b598084

mentioned in issue #166 (closed)

mentioned in issue #54