hyperledger / anoncreds-rs

anoncreds-rs
https://wiki.hyperledger.org/display/anoncreds
Apache License 2.0
73 stars 51 forks source link

Numbers should be included as numbers in the AnonCreds W3C VC #297

Closed TimoGlastra closed 5 months ago

TimoGlastra commented 6 months ago

I think they are currently included as string values, but this means we can't use DIF PE number filtering on it (like age > 18).

Would it be possible to include it as a number if the value is already a number?

cc @swcurran @Artemkaaas

TimoGlastra commented 6 months ago

@Artemkaaas Could you maybe take a look at this one as well? I think it's the last one to make the implementation align with the test vectors