Thanks for the PR, please do not integrate it yet. This code will be part of Duniter 1.7.x.
Understood :)
Created by: coveralls
Coverage increased (+0.02%) to 79.315% when pulling 5f8f3b78 on typage into f7797fcc on dev.
Coverage increased (+0.07%) to 79.349% when pulling 86ba2884 on typage into 4c67e0e8 on dev.
It's ready to merge
I will have a look today, there are a few things I need to check.
It seems all OK to me, except the sandbox which needs to keep control over the issuer field of an identity, a certification and a membership, which is not issuers but pubkey, from and issuer.
issuers
pubkey
from
issuer
I just wait the CI tests and merge the PR.
I was wrong, your code was already good! Sorry for polluting
yes I have modified the logic so that any Duniter document is only a table of issuers, I find it more elegant
