PolymeshAssociation / Polymesh

Node for Polymesh Blockchain
https://polymesh.network/
GNU General Public License v3.0
165 stars 47 forks source link

MESH-1873/ Identity.revoke_claim doesn't check if the claim exists #1361

Closed JMoore96 closed 1 year ago

JMoore96 commented 1 year ago

changelog

modified logic

codecov[bot] commented 1 year ago

Codecov Report

Merging #1361 (36e35f8) into develop (17a8cd5) will decrease coverage by 0.00%. The diff coverage is 50.00%.

:exclamation: Current head 36e35f8 differs from pull request most recent head 75cc335. Consider uploading reports for the commit 75cc335 to get more accurate results

@@             Coverage Diff             @@
##           develop    #1361      +/-   ##
===========================================
- Coverage    43.56%   43.55%   -0.01%     
===========================================
  Files          140      140              
  Lines        30108    30111       +3     
===========================================
- Hits         13117    13116       -1     
- Misses       16991    16995       +4     
Impacted Files Coverage Δ
pallets/identity/src/lib.rs 15.91% <0.00%> (-0.05%) :arrow_down:
pallets/identity/src/claims.rs 90.80% <100.00%> (+0.02%) :arrow_up:
pallets/identity/src/keys.rs 80.68% <0.00%> (-0.16%) :arrow_down:
pallets/balances/src/lib.rs 57.10% <0.00%> (-0.14%) :arrow_down: