TBD54566975 / tbdex

56 stars 25 forks source link

Specify null case for private data hashes #313

Closed diehuxx closed 4 months ago

diehuxx commented 4 months ago

Update spec for Rfq.data and Rfq.privateData to make Rfq.data.claimsHash optional. claimsHash is already optional on tbdex-js, tbdex-kt, tbdex-swift, and tbdex-dart. This PR brings the spec into alignment with the intention and existing implementations.

Also, add spec text clarifying that expected behavior when Rfq.data.*Hash is null/omitted, then the corresponding Rfq.privateData.* field must also be null/omitted.

This PR contains a change to JSON schema: