search

TBD54566975 / web5-spec

https://tbd54566975.github.io/sdk-report-runner/
Apache License 2.0
6 stars 5 forks source link

portable did metadata could be optional. #152

Closed AlexHsieh closed 3 months ago

AlexHsieh commented 3 months ago

PortableDid in web5-js is optional PortableDid in web5-kt is NOT optional PortableDid in web5-swift is optional

KT version might require update?

nitro-neal commented 3 months ago

Thank you for this, So it seems that portableDid metadata is not optional in web5-js not optional in web5-kt

image image

So I think swift needs to be changed to have metadata not optional.

I could be missing some context here so please let me know if I am off the mark

kirahsapong commented 3 months ago

This was a good catch @AlexHsieh, thanks for tracking it down!

Just opened an issue in web5-swift to make this field required.

Thanks @nitro-neal! Closing this PR.