Skip to content
This repository has been archived by the owner on Mar 27, 2024. It is now read-only.

Optional credential manifest fields are not omitted #3335

Closed
HeidiHan0000 opened this issue Aug 15, 2022 · 0 comments · Fixed by #3336
Closed

Optional credential manifest fields are not omitted #3335

HeidiHan0000 opened this issue Aug 15, 2022 · 0 comments · Fixed by #3336
Assignees

Comments

@HeidiHan0000
Copy link
Contributor

There are several properties of credential manifests, credential fulfillments, and credential applications which are optional as per DIF specs. These properties were defined in the structs with the omitempty tag but were not actually omitted because there are no default empty values for embedded structs. This affected some validate functions as the assumption that certain fields do not exist were incorrect.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging a pull request may close this issue.

1 participant